portal_tabs translations do not seem to work for @navigation endpoint.
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 45/100
Research direction
Start in src/plone/restapi/services/navigation/get.py, focusing on how portal_tabs entries are converted into navigation responses and when the Home tab translation branch is reached. Verify the @navigation endpoint with a translated portal and ensure the Home tab title is translated in the response.
Written by the indexing model from the issue text.
Description
Hello, in here
https://github.com/plone/plone.restapi/blob/main/src/plone/restapi/services/navigation/get.py
we have the following snippet:
for tab in self.portal_tabs:
entry = {}
entry.update(
{
"path": "/".join((navtree_path, tab["id"])),
"description": tab["description"],
"@id": tab["url"],
}
)
if "review_state" in tab:
entry["review_state"] = json_compatible(tab["review_state"])
else:
entry["review_state"] = None
if "title" not in entry:
entry["title"] = tab.get("name") or tab.get("description") or tab["id"]
else:
# translate Home tab
entry["title"] = translate(
entry["title"], domain="plone", context=self.request
)
entry["title"] = safe_text(entry["title"])
ret[navtree_path].append(entry)
To me it seems we can never reach the "translate" path so in Volto the "Home" tab does not get translated.
Best,
Erkka
- Dominant language
- Python
- Stars
- 109
- Forks
- 107
- Avg merge
- 2d 3h
- Merged PRs (30d)
- 4
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from plone/plone.restapi
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
plone/plone.restapi#2031 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
plone/plone.restapi#662 · 2 comments · 4 reactions ·
-
01 type: bug
plone/plone.restapi#2023 · 1 comment · 1 assignee ·
-
Difficulty 5/5 Over a week Newbie friendliness 25/100
plone/plone.restapi#2017 ·
-
@controlpanels/content-rules/<rule>/action/<idx> omits values for fields in non-default fieldsets Open
plone/plone.restapi#2016 · 1 assignee ·
All issues in plone/plone.restapi
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
bancolombia/sentinel#23 ·
-
test md OpenCI
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
-
integration:quickjs org:external priority:backlog topic:code-interpreter topic:middleware type:feature
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
langchain-ai/deepagents#6450 ·
-
bug client
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100