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-1168] Update access policy selector to disable on submit #8519
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #8519 +/- ##
==========================================
- Coverage 27.70% 27.69% -0.01%
==========================================
Files 2392 2392
Lines 69368 69375 +7
Branches 12938 12938
==========================================
Hits 19215 19215
- Misses 48673 48680 +7
Partials 1480 1480 ☔ View full report in Codecov by Sentry. |
No New Or Fixed Issues Found |
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
Type of change
Objective
The purpose of this PR is to update usage of the access policy selector to disable the form on submit and make loading more consistent with other tabs.
Code changes
bitwarden_license/bit-web/src/app/secrets-manager/projects/project/project-people.component.html:
Add spinner
bitwarden_license/bit-web/src/app/secrets-manager/projects/project/project-people.component.ts:
Fix floating promises.
Disable the form group on submit.
bitwarden_license/bit-web/src/app/secrets-manager/service-accounts/people/service-account-people.component.html:
Add spinner
bitwarden_license/bit-web/src/app/secrets-manager/service-accounts/people/service-account-people.component.ts:
Fix floating promises.
Remove catch error since this is now covered by
serviceAccountAccessGuard
Disable the form group on submit.
bitwarden_license/bit-web/src/app/secrets-manager/shared/access-policies/access-policy-selector/access-policy-selector.component.html:
Add disabled to required fields.
Screenshots
Before Project → People load in
After Project → People load in
Before Project → People submit
After Project → People submit
Before Service Account → People load in
After Service Account → People load in
Before Service Account → People submit
After Service Account → People submit
Before you submit