Skip to content

Conversation

@hexaltation
Copy link
Collaborator

@hexaltation hexaltation commented Mar 18, 2025

Context

Some strings are not translated in AdminPanel.

  • Some strings are not prefixed with the good scope
  • Some strings are not passed to i18n t() function in the code

Proposed solution

This PR only does partially it only fixes client side strings.

Out of scope

Some more job has to be done as some chains are generated server side, at least in app/server/lib/BootProbes.ts
And it seems to use some keys of objects as string to be translated, which will need some more job.

Has this been tested?

  • 👍 yes, new strings are in `static/locales/en.client.json

Copy link
Collaborator

@manuhabitela manuhabitela left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Grégoire :)

@hexaltation hexaltation moved this from Needs feedback to In Progress in French administration Board Apr 9, 2025
@hexaltation hexaltation moved this from In Progress to Todo in French administration Board Jun 18, 2025
@fflorent fflorent moved this from Todo to In Progress in French administration Board Aug 12, 2025
@hexaltation hexaltation moved this from In Progress to Needs Internal Feedback in French administration Board Aug 13, 2025
@fflorent fflorent force-pushed the admin-panel-translation branch from b4215c8 to e4b66b9 Compare August 13, 2025 14:50
@hexaltation hexaltation force-pushed the admin-panel-translation branch from de79d6e to addb8bd Compare August 14, 2025 12:39
Copy link
Collaborator

@manuhabitela manuhabitela left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @hexaltation 🤝

@hexaltation hexaltation force-pushed the admin-panel-translation branch from ad6b75c to 7af847e Compare August 19, 2025 09:13
@RaywanS RaywanS moved this from Needs Internal Feedback to Needs feedback in French administration Board Aug 20, 2025
@hexaltation hexaltation force-pushed the admin-panel-translation branch from 7af847e to 2b2d4a2 Compare August 25, 2025 16:04
hexaltation and others added 6 commits September 2, 2025 17:45
This PR only does partially it only fixes client side strings.
Some more job has to be done as some chains are generated
server side, at least in `app/server/lib/BootProbes.ts`
@hexaltation hexaltation force-pushed the admin-panel-translation branch from 87da085 to 027eee9 Compare September 2, 2025 15:45
@paulfitz paulfitz merged commit 3355462 into gristlabs:main Sep 2, 2025
14 checks passed
@github-project-automation github-project-automation bot moved this from Needs feedback to Done in French administration Board Sep 2, 2025
@hexaltation hexaltation deleted the admin-panel-translation branch November 4, 2025 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants