-
Notifications
You must be signed in to change notification settings - Fork 0
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
Added Logout button #61
Conversation
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.
Nice work! Looks like its working as-is. I provided an alternative option where you don't have to create a custom devise route. I think it is a cleaner approach. But I will let you and @camckin10 decide.
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 looks great Kenyon!
Thank you @edithemmings for the suggestion about cleaning up the scope code.
Before you merge please:
--resolve all conversations on your PR
--Update your ticket description to include that you were able to make the route work.
--Change your PR from draft to ready to review.
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.
👍
Changes
Give a short summary of the changes made in this PR.
Link to Issue
Link Asana ticket here!
Comments, Notes, or Questions
I added the button to the sidebar. Now I am working on getting it to redirect to the sign in page.
Related Screenshots
For FE tickets specifcially, please include a screenshot of your work!
routes are working.
Before you submit for review, Did you...