-
Notifications
You must be signed in to change notification settings - Fork 111
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
LG-12030 Remove 'create' action from BackupCodeSetupController #10088
Conversation
437cf59
to
79b571d
Compare
67d44af
to
146cb85
Compare
e7e1fde
to
51033bc
Compare
0c3605c
to
871c65f
Compare
51033bc
to
fa6c393
Compare
c4bc096
to
0c12d42
Compare
906c986
to
92c9c52
Compare
@@ -280,7 +280,6 @@ | |||
get '/backup_code_refreshed' => 'users/backup_code_setup#refreshed' | |||
get '/backup_code_reminder' => 'users/backup_code_setup#reminder' | |||
get '/backup_code_setup' => 'users/backup_code_setup#index' | |||
patch '/backup_code_setup' => 'users/backup_code_setup#create', as: :backup_code_create |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like we still have something directing to this page, as production logs show activity for PATCH /backup_code_setup
, so this is not safe to remove yet. I'll investigate.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be fixed as of #10464, but will confirm once those changes go live in RC 374.
changelog: User-Facing Improvements, MFA, Remove backup code pre-warning from setup flow
4eb5b28
to
3313d1c
Compare
I squashed and rebased the branch to resolve conflicts and bring this up to date with main. |
changelog: User-Facing Improvements, MFA, Remove backup code pre-warning from setup flow
馃帿 Ticket
LG-12030
馃洜 Summary of changes
Follow on PR for #10014
Removes create action, route and view. Update specs accordingly.