New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
I18n: Translate phrases for new Browse Dashboards #70654
I18n: Translate phrases for new Browse Dashboards #70654
Conversation
We'll check this more in the week, but to enable a feature flag your grafana server config (custom.ini) should look like this
Also be sure to run |
17af39e
to
68cd489
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Bohdanator thank you for your contribution, please check the comments I left for you.
? t('browse-dashboards.action.deleting', 'Deleting...') | ||
: t('browse-dashboards.action.delete-button', 'Delete'), | ||
[isDeleting] | ||
); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think the useMemo hook is necessary in this case, you can just use the t() function inside the ternary directly
() => | ||
isMoving ? t('browse-dashboards.action.moving', 'Moving...') : t('browse-dashboards.action.move-button', 'Move'), | ||
[isMoving] | ||
); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See above comment about useMemo hook.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, we merged main and resolved some merge conflicts , we had some phrases that were already translated and we went with what was already in main. Thank you for contributing.
* Mark strings in folder manager nav for translation * Mark strings in folder actions for translation * Mark strings in new folder modal for translation * Mark strings in delete modal for translation * Mark strings in move modal for translation * Fix interpunction in translations * Run i18n:extract * Fix Manage Permissions drawer test * Redundnt useMemo * Updated extracted translations after resolving conflicts --------- Co-authored-by: Roxana Turc <anamaria-roxana.turc@grafana.com>
What is this feature?
This PR marks strings in the new folder management UI for translation.
Which issue(s) does this PR fix?:
Fixes #70085
Special notes for your reviewer:
I am completely new to this project :). I was able to set up Grafana locally, but I wasn't able to toggle
nestedFolders
feature. After putting it incustom.ini
config file nothing happened (also after restart). What else is required?I would also like feedback on the translation IDs. This is my first time working with
i18n
or Crowdin, so I'm not sure what else is required for these translation IDs to be added. (Also if the ID structure makes sense.)Please check that: