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
CONSOLE-2447: i18n for secrets #7047
Conversation
/assign @spadgett |
11bc44d
to
b0cf2c3
Compare
/retest |
b0cf2c3
to
e0b5fde
Compare
/retest |
e0b5fde
to
ef4bba0
Compare
ef4bba0
to
bea73a5
Compare
bea73a5
to
e783eb3
Compare
/retest |
1 similar comment
/retest |
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.
/lgtm
/hold for approvals
/assign @yapei @ahardin-rh @sferich888
/retest |
e783eb3
to
5e2d2d0
Compare
@spadgett - had to rebase due to json conflict |
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.
/lgtm
/retest |
2 similar comments
/retest |
/retest |
/hold It looks like the secret tests aren't passing. |
@spadgett - Been working on this all morning; the tests all pass locally so I am not sure what if happening. |
2127d4f
to
8bb47a0
Compare
@spadgett - Fixed the test cases and rebased. |
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, one small Cypress fix -thanks
8bb47a0
to
f82ac67
Compare
f82ac67
to
4bd6185
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.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dtaylor113, spadgett, zherman0 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 |
/retest |
1 similar comment
/retest |
/retest So far we have had annotation flakes and a variety of others. |
/hold cancel |
/retest |
/retest Please review the full test history for this PR and help us cut down flakes. |
CONSOLE-2447: i18n for secrets
Needs #7045 to merge to be complete.
Also includes a missed translation from the details page component that was found during my testing.