Skip to content
This repository was archived by the owner on Mar 15, 2024. It is now read-only.

Conversation

@kspearrin
Copy link
Member

Type of change

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

Objective

Apply client updates from bitwarden/jslib#753

Code changes

React to changes from bitwarden/jslib#753

Also, added a "learn more" icon for username types on the generator.

NOTE: I still need to update jslib once bitwarden/jslib#753 is merged.

Before you submit

  • I have checked for linting errors (npm run lint) (required)
  • This change requires a documentation update (notify the documentation team)
  • This change has particular deployment requirements (notify the DevOps team)

@kspearrin kspearrin requested a review from a team March 31, 2022 16:19
Copy link
Member

@eliykat eliykat left a comment

Choose a reason for hiding this comment

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

Looks good pending jslib

eliykat
eliykat previously approved these changes Apr 1, 2022
@kspearrin kspearrin requested a review from eliykat April 1, 2022 03:27
@kspearrin kspearrin merged commit cb8a40d into master Apr 1, 2022
@kspearrin kspearrin deleted the genupdate branch April 1, 2022 03:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants