Skip to content

Resolve "Cleanup in the welcome controller"

Roy Liu requested to merge 391998-cleanup-in-the-welcome-controller into master

What does this MR do and why?

Remove the line that return users_almost_there_path. It's discovered during the discussion here: !107302 (comment 1283442858) that while email_confirmation_setting is hard, it's not possible for an unconfirmed user to be redirected back to users_almost_there_path, the user will be redirected to login always.

This is also reflected in the spec here: https://gitlab.com/gitlab-org/gitlab/-/blob/master/spec/controllers/registrations/welcome_controller_spec.rb#L91

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #391998 (closed)

Edited by Roy Liu

Merge request reports

Loading