Skip to content
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

Prepare for moving the RecoveryCodes component to enterprise #41464

Merged
merged 1 commit into from
May 15, 2024

Conversation

bl-nero
Copy link
Contributor

@bl-nero bl-nero commented May 13, 2024

Allow the call site to specify component that will be displayed in the welcome flow and add some icons required to implement the new UI on the Enterprise side. Pure refactoring, no visual changes expected.

I'm going to move this component because I need to change it, and it will be easier to avoid build breakages due to snapshot test changes if it's in the enterprise repo.

Contributes to #37615

Allow the call site to specify component that will be displayed in the welcome
flow and add some icons required to implement the new UI on the Enterprise
side.
@bl-nero bl-nero added no-changelog Indicates that a PR does not require a changelog entry backport/branch/v15 labels May 13, 2024
@bl-nero
Copy link
Contributor Author

bl-nero commented May 13, 2024

Cases tested:

  • Welcome flow, OSS
  • Reset flow, OSS
  • Welcome flow, Enterprise Cloud
  • Reset flow, Enterprise Cloud
  • Recovery flow, Enterprise Cloud
  • Regenerating reset codes, Enterprise Cloud

@bl-nero bl-nero added this pull request to the merge queue May 15, 2024
Merged via the queue into master with commit 1297b5f May 15, 2024
39 of 40 checks passed
@bl-nero bl-nero deleted the bl-nero/recovery-codes branch May 15, 2024 09:45
@public-teleport-github-review-bot

@bl-nero See the table below for backport results.

Branch Result
branch/v15 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/branch/v15 no-changelog Indicates that a PR does not require a changelog entry size/sm ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants