Skip to content
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

Logout in navbar, admin only edit users #18

Merged
merged 6 commits into from
Apr 25, 2023
Merged

Logout in navbar, admin only edit users #18

merged 6 commits into from
Apr 25, 2023

Conversation

MAPReiff
Copy link
Owner

@MAPReiff MAPReiff commented Apr 23, 2023

Modified the temp nav bar to include a logout link

  • modified the Return to Homepage link to be in the main template along with the new logout link

Modified routes for users/edit so GET/POST only work for admins
Modified the page titles for users/view and users/edit
Modified login/register pages so that when there is a server-side error the page updates properly.

@MAPReiff MAPReiff added enhancement New feature or request routes Changes to the routes directory files users Changes to users pages Changes to pages (HTML, CSS, Handlebars, etc) labels Apr 23, 2023
@MAPReiff
Copy link
Owner Author

Rebase with main and resolved merge conflicts

@dbajollari1 dbajollari1 merged commit 3f9a318 into main Apr 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request pages Changes to pages (HTML, CSS, Handlebars, etc) routes Changes to the routes directory files users Changes to users
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants