Update: User Registration Redirect Path #1290
Conversation
|
@Gabe-Torres how's this going? Let me know if you have any questions or would like to re-assign! |
|
Hey @kasugaijin! Going well. Happy new year! I can still complete this ticket. Did you get a chance to look at my video above? I have the issue solved but was wondering if there's a different way to go about it using devise that you know of? I can merge the most recent main and get it ready for closing if everything looks good 👍 |
|
@Gabe-Torres let me take a look tomorrow! |
| @@ -56,6 +56,19 @@ def account_update_params | |||
| :avatar) | |||
| end | |||
There was a problem hiding this comment.
Nice. I think this is a lot easier to read!
|
Just one test likely needs updating |
I'll get that test updated and merge the most recent main! 👍🏽 |
Gabe-Torres
force-pushed
the
1227-Edit-User-Registration-Redirect-Path
branch
from
9e94986 to
4b6dd0f
Compare
|
hmmm... WELL everything was passing, rebased main into my branch and it all looked good to go.. but I wanted to clean up the history and failed big it seems I wanted to avoid making another branch like the previous issues I've taken. Not quite sure what happen, I'll wait for some feedback before I proceed |
|
Doh! |
|
Don't think I broke anything in the process but the branch might be lost 😅 |
🔗 Issue
#1227
✍️ Description
This update utilizes Devise's after_update_path_for method to handle user redirection after updating their registration.
If the user is part of the staff roster, associated with a valid organization, and authorized to access the organization's dashboard, they will be redirected to their staff dashboard.
If the user is an adopter or fosterer and authorized to access the adopter/foster dashboard, they will be redirected there.
If neither condition is met, the user will be redirected to the root path.
Adds: Controller tests for user registration updates: Test case for adopter/fosterer user redirection.
Test case for staff user redirection.
📷 Screenshots/Demos
(https://jam.dev/c/62b85dbb-12bc-4248-b63d-936d9f562b16)