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 ocs dashboards #7368
i18n ocs dashboards #7368
Conversation
@rawagner changes for review graph-empty inventory dashboard.ts details-item monitoring Thanks! |
@@ -29,7 +30,7 @@ export const healthStateMapping: { [key in HealthState]: HealthStateMappingValue | |||
priority: 1, | |||
health: HealthState.UNKNOWN, | |||
icon: <GrayUnknownIcon title="Unknown" />, | |||
message: 'Unknown', | |||
message: i18next.t('dashboard~Unknown'), |
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.
can you add i18n for all states ?
I think we should convert healthStateMapping to function and pass t
as param to make sure we get the correct translation when user changes the language
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.
Sure, sounds fair! We can change this map -> function
This will need some adjustments at other places therefore I would raise another PR for ease in handling.
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.
Removed this chnage from this PR
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.
And Created another PR #7372
0241cf7
to
67cb82d
Compare
/hold |
67cb82d
to
f8133d7
Compare
/hold cancel |
d589d29
to
12412e8
Compare
12412e8
to
387cea6
Compare
/test e2e-gcp-console |
/lgtm |
/test e2e-gcp-console |
/retest Please review the full test history for this PR and help us cut down flakes. |
387cea6
to
7bc8d2d
Compare
/lgtm |
/retest Please review the full test history for this PR and help us cut down flakes. |
2 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
- independent mode dashboard - internal mode dashboard - object service dashboard Additional modifications: - dropdown options format to be {name: /*the string to be displayed and used for i18n*/,id: /*a unique name of the option-key*/} on all ocs dashboards - unit test adjustments - i18n of empty states in core components of details-card,inventory-card,graph-empty Namespaces added: ceph-storage-plugin, noobaa-storage-pugin Signed-off-by: Afreen Rahman <afrahman@redhat.com>
7bc8d2d
to
538c538
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: a2batic, afreen23, cloudbehl, rawagner The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Additional modifications:
Namespaces added: ceph-storage-plugin, noobaa-storage-pugin
cc @rawagner @bipuladh @cloudbehl @a2batic