Merge ProfNavBar and AdminNavBar into one

SENG499-COMPANY4 / frontend

A course scheduler created for the University of Victoria Software Engineering department.

http://seng-scheduler.web.app

3 stars 3 forks source link

Merge ProfNavBar and AdminNavBar into one #15

Closed markmotet closed 1 year ago

markmotet commented 1 year ago

I merged the two navbars into one that uses conditional rendering. You can set the variables in index.js on this line to cause different elements to show up:

<Navbar isLoggedIn={true} isAdmin={true} isProfessor={false} />

vijaivir commented 1 year ago

These are some nice additions, good work!

The navigation bar for the professor page is kinda repeating, can you make changes so that the set preferences and the view schedule are all on the main navbar?

markmotet commented 1 year ago

Whoops! Forgot to remove that one. It should be gone now.

vijaivir commented 1 year ago

It was still showing up on the preferences page so I've just removed that component. Should be good to merge.