-
Notifications
You must be signed in to change notification settings - Fork 1
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 Features #114
Admin Features #114
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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 LGTM but I'd wait for another review before merging 👹
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.
I like that you un-hardcoded the mobile navbar height
Co-authored-by: Alex Zhang <[email protected]>
Info
Initial Scaffolding for Admin Home Page
this will be the next biggest set of features to complete once everything user facing is done (can be completely after releasing v2 publicly) - most urgent task admin side will probably be editing and managing store items to upload multiple item images
the goal of this PR is just to set a decent foundation wiring up all links and actions (will need major styling revamps later on)
Description
Type of Change
expected)
linting/formatting)
workflows)
Testing
I have tested that my changes fully resolve the linked issue ...
Checklist
src/lib
functions and commented hard to understand areasanywhere else.
Screenshots
Please include a screenshot of your Cypress testing suite passing successfully.
If you made any visual changes to the website, please include relevant screenshots below.