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
SM-1159: Rename Service Accounts to Machine Accounts #8444
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #8444 +/- ##
=======================================
Coverage 27.15% 27.15%
=======================================
Files 2330 2330
Lines 67945 67943 -2
Branches 12693 12691 -2
=======================================
Hits 18448 18448
+ Misses 48103 48101 -2
Partials 1394 1394 ☔ View full report in Codecov by Sentry. |
apps/web/src/app/billing/organizations/sm-adjust-subscription.component.html
Show resolved
Hide resolved
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.
Looks good to me but shouldn't we delete the old i18n entries if they're not used anymore?
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.
Looks goos for billing
New Issues
|
@r-tome for now we want to keep the old entries. The main reason for this is if we want to switch back to service accounts, this will be easy, especially as it means we can also keep the translations. |
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!
Merged branch |
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
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.
Billing changes look good!
0cc09f3
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.
Reviewed AC owned file.
Note: QA will test on |
Type of change
Objective
Rename Service Accounts to Machine Accounts. In this PR we create new i18n entries, preserving the Service Account ones purposely for easy rollback and to keep the translations. Please note, this refactor only applies to text that is visible to the user, for now the code base still refers to these types of accounts as Service Accounts.
Server PR: bitwarden/server#3958
Code changes
Only added new i18n entries and swapped the old ones for the new ones. Everything should be the same except Service Account becomes Machine Account, with the casing kept based on the initial message.
Before you submit