Skip to content

upcoming: [M3-7839] - Change Business Partner to Parent User#10259

Merged
jaalah-akamai merged 6 commits intolinode:developfrom
jaalah-akamai:M3-7839
Mar 6, 2024
Merged

upcoming: [M3-7839] - Change Business Partner to Parent User#10259
jaalah-akamai merged 6 commits intolinode:developfrom
jaalah-akamai:M3-7839

Conversation

@jaalah-akamai
Copy link
Contributor

@jaalah-akamai jaalah-akamai commented Mar 5, 2024

Description 📝

We're making a copy change to the way we represent the reseller.

Changes 🔄

  • Change Business Partner to Parent User

Target release date 🗓️

3/18

Preview 📷

Screenshots
Screenshot 2024-03-05 at 10 43 07 AM Screenshot 2024-03-05 at 10 42 39 AM
Screenshot 2024-03-05 at 10 42 31 AM Screenshot 2024-03-05 at 10 52 28 AM

How to test 🧪

Prerequisites

(How to setup test environment)

  • Update parent to child in MSW

Verification steps

As an Author I have considered 🤔

Check all that apply

  • 👀 Doing a self review
  • ❔ Our contribution guidelines
  • 🤏 Splitting feature into small PRs
  • ➕ Adding a changeset
  • 🧪 Providing/Improving test coverage
  • 🔐 Removing all sensitive information from the code and PR description
  • 🚩 Using a feature flag to protect the release
  • 👣 Providing comprehensive reproduction steps
  • 📑 Providing or updating our documentation
  • 🕛 Scheduling a pair reviewing session
  • 📱 Providing mobile support
  • ♿ Providing accessibility support

@jaalah-akamai jaalah-akamai self-assigned this Mar 5, 2024
@jaalah-akamai jaalah-akamai requested review from a team as code owners March 5, 2024 16:03
@jaalah-akamai jaalah-akamai requested review from carrillo-erik, jdamore-linode and mjac0bs and removed request for a team March 5, 2024 16:03
@github-actions
Copy link

github-actions bot commented Mar 5, 2024

Coverage Report:
Base Coverage: 81.35%
Current Coverage: 81.35%

Copy link
Contributor

@mjac0bs mjac0bs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • We're missing one more reference to "business partner settings" in a comment in user-verification-banner.spec.ts, L228.
  • What do you think about replacing the string literals of "business partner" with the constant in every component that it's used? We'd have to use CSS/a util to get the capitalization right in a couple of places, but it would reinforce consistency and save us time if the language gets changed in the future. (Maybe that's not a concern. But I'm a little skeptical since there have been so many terms used to refer to the different users throughout this project.)

@mjac0bs mjac0bs added the Add'tl Approval Needed Waiting on another approval! label Mar 5, 2024
@bnussman-akamai bnussman-akamai added Approved Multiple approvals and ready to merge! and removed Add'tl Approval Needed Waiting on another approval! labels Mar 5, 2024
@jaalah-akamai jaalah-akamai merged commit f8ce5e6 into linode:develop Mar 6, 2024
vrajesh73 added a commit to vrajesh73/manager that referenced this pull request Mar 12, 2024
…eature/namespace-create

* 'develop' of https://github.com/vrajesh73/manager: (89 commits)
  fix: [M3-7269] - Display parent email in user menu when no company name is available for restricted parent user (linode#10248)
  fix: [M3-7817] - Show correct status of Child Account Enabled column for parent users (linode#10233)
  upcoming: [M3-7616] - Add Placement Groups Events and Notifications (linode#10221)
  upcoming: [M3-7816-v2] - Adjust logic for when to show Switch Account button (linode#10266)
  fix: [M3-7831] - Persisting error messages in ACLB delete dialogs (linode#10254)
  upcoming: [M3-7842] - Update Assign Linode Drawer and improve query skipping (linode#10263)
  upcoming: [M3-7704] - Disable Cloning, Private IP, Backups for edge regions (linode#10222)
  test: Fix test flake for Images landing page test (linode#10267)
  fix: [M3-7824] - ACLB TCP Rule Creation and other fixes (linode#10264)
  refactor: [M3-7687] - Linodes Restricted User Experience 2/2 (linode#10227)
  test: Resolve OBJ create and delete E2E test flake (linode#10245)
  upcoming: [M3-7723] - Placement Group feature flag as object (linode#10256)
  chore(deps): Bump sanitize-html from 2.11.0 to 2.12.1 (linode#10247)
  change: [M3-7813] - Allow the disabling of the TypeToConfirm input (linode#10251)
  upcoming: [M3-7839] - Change Business Partner to Parent User (linode#10259)
  upcoming: [M3-7835] - Adjust user table column count (linode#10252)
  upcoming: [M3 7738] - Update Placement Group Create & Edit Drawers (linode#10205)
  refactor: [M3-7437] - Use `@lukemorales/query-key-factory` for Profile Queries (linode#10241)
  fix: React Query `updateInPaginatedStore` helper function not working as expected (linode#10249)
  test: [M3-7497] - Add tests for child user verification banner (linode#10204)
  ...

# Conflicts:
#	packages/manager/src/MainContent.tsx
#	packages/manager/src/dev-tools/FeatureFlagTool.tsx
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved Multiple approvals and ready to merge! Parent / Child Account

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants