Closed Nael-Nathanael closed 3 years ago
So I think we probably won't merge this until we have some interface for administrators on the website. Perhaps we shouldn't even have the admin login as a featured thing on the navbar and just leave it as a separate routed page
One other minor nitpick is that all the other navbar icons are darkened when not in focus. This can probably be fixed fairly easily if we just add a dedicated administrator page instead of a popup, but again, I don't know that we want to advertise that this page has an admin page
I understand, I'll do some changes and create another PR in the future...
As I've mentioned in #162 about frontend authorization, I've add several component and functions to connect with backend's JWT and store it on user's session storage.