[AC-2420] Hide SM checkbox on member invite when org is on SM Standalone

[amorask-bitwarden]

Type of change

- [ ] Bug fix
- [X] New feature development
- [ ] Tech debt (refactoring, code cleanup, dependency upgrades, etc)
- [ ] Build/deploy pipeline (DevOps)
- [ ] Other

Objective

This PR updates the people.component to check if the current organization is on SM Standalone. If it is, when the user tries to invite a new member, it hides the option to give the invited member access to SM and defaults it to true. This enforces a rule that SM standalone orgs cannot invite PM only members.

Code changes

• ...admin-console/organizations/members/components/member-dialog/member-dialog.component.html: Only show SM checkbox if the organization isn't on SM standalone.
• ...p/admin-console/organizations/members/components/member-dialog/member-dialog.component.ts: Default accessSecretsManager to true when org is on SM standalone.
• apps/web/src/app/admin-console/organizations/members/people.component.ts: Figure out if org is on SM standalone and pas that info to the invite member modal.
• libs/common/src/billing/services/billing-api.service.ts: Bring a GET organization subscription method under billing ownership.
• libs/common/src/billing/services/organization-billing.service.ts: Add helper method to determine if org is on SM standalone.

Unrelated Refactoring

• ...eb/src/app/billing/providers/clients/manage-client-organization-subscription.component.ts: Rename to be more accurate.
• libs/common/src/billing/models/request/provider-organization-update.request.ts: Rename to be more accurate.

Screenshots

Screen.Recording.2024-04-08.at.2.51.23.PM.mov

Before you submit

• Please add unit tests where it makes sense to do so (encouraged but not required)
• If this change requires a documentation update - notify the documentation team
• If this change has particular deployment requirements - notify the DevOps team
• Ensure that all UI additions follow WCAG AA requirements

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 18 lines in your changes are missing coverage. Please review.

Project coverage is 27.31%. Comparing base (a45706c) to head (fd737e8).

Files	Patch %	Lines
...c/billing/services/organization-billing.service.ts	0.00%	8 Missing ⚠️
...-console/organizations/members/people.component.ts	0.00%	4 Missing ⚠️
...omponents/member-dialog/member-dialog.component.ts	0.00%	3 Missing ⚠️
...common/src/billing/services/billing-api.service.ts	0.00%	3 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #8644      +/-   ##
==========================================
- Coverage   27.32%   27.31%   -0.01%     
==========================================
  Files        2349     2349              
  Lines       68613    68632      +19     
  Branches    12828    12831       +3     
==========================================
  Hits        18748    18748              
- Misses      48455    48474      +19     
  Partials     1410     1410              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[github-actions]

Checkmarx One – Scan Summary & Details – b263f183-1baf-4327-9fda-2310fe697a80

No New Or Fixed Issues Found

[r-tome]

LGTM!

[amorask-bitwarden]

@cturnbull-bitwarden @r-tome Can I get another look at this when you both have a sec? I had to change it from hiding the SM checkbox to defaulting it to checked and disabling it as shown below: