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

fix(is-valid-autocomplete): allow autocomplete="one-time-code" #2336

Merged
merged 1 commit into from
Jun 30, 2020

Conversation

lencioni
Copy link
Contributor

@lencioni lencioni commented Jun 30, 2020

This autocomplete value can be used by some browsers to automatically
fill in the value after receiving a one-time-code for 2-factor
authentication. More info:

Closes issue: #2335

Reviewer checks

Required fields, to be filled out by PR reviewer(s)

  • Follows the commit message policy, appropriate for next version
  • Code is reviewed for security

@lencioni lencioni requested a review from a team as a code owner June 30, 2020 15:29
@CLAassistant
Copy link

CLAassistant commented Jun 30, 2020

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

@straker straker left a comment

Choose a reason for hiding this comment

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

@straker
Copy link
Contributor

straker commented Jun 30, 2020

Just need to sign the license/cla and we get this merged in.

@faroelrozi
Copy link

faroelrozi commented Jun 30, 2020 via email

@lencioni
Copy link
Contributor Author

@straker CLA signed, thanks!

@straker straker changed the title Allow autocomplete="one-time-code" fix(is-valid-autocomplete): allow autocomplete="one-time-code" Jun 30, 2020
@straker straker merged commit 638346f into dequelabs:develop Jun 30, 2020
@lencioni lencioni deleted the one-time-code branch July 1, 2020 13:11
@backwardok
Copy link

@straker Did this get added to a release yet? I looked at 4.0.1 and 4.0.2, which have both released since this PR was merged, from the changelog, but I didn't see this change.

@straker
Copy link
Contributor

straker commented Oct 5, 2020

@backwardok Yep, should have made it into the 4.0.0 release.

@backwardok
Copy link

Great, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants