Skip to content

fix: register the app menu entry in PHP so "Office" isn't translated - #200

Open
juliusknorr with Copilot wants to merge 1 commit into
mainfrom
copilot/fix-translated-app-name
Open

juliusknorr with Copilot wants to merge 1 commit into
mainfrom
copilot/fix-translated-app-name

Conversation

Copilot AI commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

The navigation name in info.xml is extracted to Transifex, so the product name "Office" got translated (e.g. "Büro"). The <navigations> block is removed and the entry is now registered via INavigationManager in a new LoadNavigationEntryListener on LoadAdditionalEntriesEvent (same approach as spreed), with an untranslated name and the same route, icon and type.

Notes:

  • order => 10 is new, the old entry had none.
  • Stale "Office" translations can only be cleaned up in Transifex, since the top-level <name> is still extracted.

Fixes #17

Copilot AI linked an issue Oct 14, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Fix app name translation to retain product name fix: register the app menu entry in PHP so "Office" isn't translated Oct 14, 2026
Copilot AI requested a review from juliusknorr October 14, 2026 18:18
@juliusknorr
juliusknorr marked this pull request as ready for review October 14, 2026 18:21
…ated

Assisted-by: GitHubCopilot:copilot-swe-agent
Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Julius Knorr <jus@bitgrid.net>
@juliusknorr
juliusknorr force-pushed the copilot/fix-translated-app-name branch from 17aaabb to 9ccce57 Compare October 15, 2026 06:37
@juliusknorr
juliusknorr requested a review from elzody October 15, 2026 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

App name is translated

2 participants