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

Admin Sidebar & Dashboard setup #23

Merged
merged 11 commits into from
Jan 25, 2024
Merged

Admin Sidebar & Dashboard setup #23

merged 11 commits into from
Jan 25, 2024

Conversation

noahgiboney
Copy link
Collaborator

@noahgiboney noahgiboney commented Jan 25, 2024

Developer: Noah Giboney

Closes #13

Pull Request Summary

Created admin sidebar and dashboard setup. Sidebar is a use client component that uses the useSegementLayout from next/navigation. This is an easier way to determine what page user is on rather than useState. I did use useState however for the collapsing slider. Adding admins options is easy and can be done once we have a better idea what data will fill the dash. Let me know if anything needs to be changed or adjusted accordingly.

Modifications

{list out the files created/modified and a brief description of what was changed}

Admin directory in app directory and admin directory in styles for CSS. Created sidebar in the components folder. In app/admin i created a layout for all admin pages and one folder for events which is an example page. Each page has its corresponding landing page.tsx.

Testing Considerations

Mobile responsiveness works but might to be updated once dashboard has content. Also I was very rarely getting a runtime error about outdate node.js and null object, might have to keep an eye on this.

Pull Request Checklist

  • Code is neat, readable, and works
  • Comments are appropriate
  • The commit messages follows our guidelines
  • The developer name is specified
  • The summary is completed
  • Assign reviewers

Screenshots/Screencast

Demo of sidebar and dashboard

Screen.Recording.2024-01-24.at.6.01.00.PM.mp4

Copy link
Collaborator

@javalosr2004 javalosr2004 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code is immaculate, congrats on being the first to merge into the repo! One thing I do see as an area of improvement is to have some padding on top of the view for the actual page, for example for the mobile view, having it match the navbar icon
image
and for the desktop view having it match where it says Admin
image
Don't feel like you need to do it though, I can always pick it up!

@javalosr2004 javalosr2004 reopened this Jan 25, 2024
@javalosr2004 javalosr2004 merged commit 9bcccc2 into hack4impact-calpoly:main Jan 25, 2024
1 of 7 checks passed
javalosr2004 added a commit that referenced this pull request Apr 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create Admin Dashboard
2 participants