<!-- Please read the contributing guidelines before making a PR: https://github.com/openstreetmap/openstreetmap-website/blob/master/CONTRIBUTING.md
Pay particular attention to the section on how to present PRs: https://github.com/openstreetmap/openstreetmap-website/blob/master/CONTRIBUTING.md#pull-requests --> ### Description Draft PR for a restful refactor of the confirmations controller. The original intention here was for a purely structural set of changes that otherwise preserved existing functionality. However, moving toward a more restful structure prompted reusing the `#show` endpoint for both confirmation scenarios, which in turn introduced a few changes in behavior. I am looking for some initial feedback that this is going in an acceptable direction. I am of course happy to break this out into smaller PRs. Changes: - Introduce restful routes for confirmations controller - Deprecate `new_user` and `new_email` tokens in favor of single `account_confirmation` token - Some UI changes around confirmation and error messages Remaining work: - [ ] Fix failing tests - [ ] Add more test coverage ### How has this been tested? I've gone through both of the new account and email change confirmation flows locally. I've also tested out the scenarios where confirmation emails are sent out with the currently deployed token generation logic but are handled by the code in this PR. You can view, comment on, or merge this pull request online at: https://github.com/openstreetmap/openstreetmap-website/pull/5895 -- Commit Summary -- * Move controller methods to user model * Add restful routes * Make controller restful * Update route invocations * Centralize token usage * Update views * Update abilities * Update strings * Get tests passing * Fix a couple of bugs -- File Changes -- M app/abilities/ability.rb (2) M app/controllers/concerns/session_methods.rb (2) M app/controllers/concerns/user_methods.rb (4) M app/controllers/confirmations_controller.rb (111) M app/controllers/users_controller.rb (4) M app/mailers/user_mailer.rb (10) M app/models/user.rb (55) D app/views/confirmations/confirm.html.erb (36) D app/views/confirmations/confirm_email.html.erb (16) A app/views/confirmations/new.html.erb (20) M config/locales/en.yml (16) M config/routes.rb (13) M test/controllers/confirmations_controller_test.rb (152) -- Patch Links -- https://github.com/openstreetmap/openstreetmap-website/pull/5895.patch https://github.com/openstreetmap/openstreetmap-website/pull/5895.diff -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5895 You are receiving this because you are subscribed to this thread. Message ID: <openstreetmap/openstreetmap-website/pull/5...@github.com>
_______________________________________________ rails-dev mailing list rails-dev@openstreetmap.org https://lists.openstreetmap.org/listinfo/rails-dev
