-
Notifications
You must be signed in to change notification settings - Fork 32
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
[ui-strings] Consistent language use applied across UI strings #1110
Conversation
Issue rancher#6407 - Base UI review performed. Signed-off-by: Alexandra Settle <asettle@suse.com>
Signed-off-by: Alexandra Settle <asettle@suse.com>
@a110605 I will not be able to review this PR any time soon. I suggest that we just apply the suggested changes and do another general review of the strings later on. |
Co-authored-by: Simon Flood
Signed-off-by: Alexandra Settle <asettle@suse.com>
* addon development guide * include feedback from Kiefer * renamed addon to add-on to better reflect upcoming ui changes: harvester/dashboard#1110 * Apply suggestions from code review include feedback for PR review. Co-authored-by: Jillian <67180770+jillian-maroket@users.noreply.github.com> * Update docs/developer/addon-development.md Co-authored-by: Alexandra Settle <asettle@suse.com> --------- Co-authored-by: Jillian <67180770+jillian-maroket@users.noreply.github.com> Co-authored-by: Alexandra Settle <asettle@suse.com>
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.
Sanity check with rancher embedded mode. LGMT.
Let's backport to release-harvester-v1.4
after merge.
Signed-off-by: andy.lee <andy.lee@suse.com>
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.
Thanks Alex! Looks good, there is a conflict with master though.
The changes in shell
directory should also be ported in Rancher.
cc @a110605
Done solve conflict. |
@mergify backport release-harvester-v1.4 |
✅ Backports have been created
|
[ui-strings] Consistent language use applied across UI string (cherry picked from commit f519a8f) # Conflicts: # pkg/harvester/l10n/en-us.yaml
…ster#1110) [ui-strings] Consistent language use applied across UI string
@mergify backport release-harvester-v1.4 |
✅ Backports have been created
|
Summary
Correct spelling mistakes and base UI review performed:
Related Issue #
Issue #6407