Skip to content

Navbar - #20

Open
RareDoge wants to merge 58 commits into
hack4impact-calpoly:developfrom
manavos:navbar
Open

RareDoge wants to merge 58 commits into
hack4impact-calpoly:developfrom
manavos:navbar

Conversation

@RareDoge

Copy link
Copy Markdown

Developer: Corey Moore

Closes #1

Pull Request Summary

The purpose of this PR is to implement the nav-bar. I updated the name of the company and put it in the nav-bar. I am open to changing the color and look of the nav-bar when we discuss it. Added a logo and some links.

Modifications

Modified globals.css so that the body has margin 0. Created file in components called Navbar which holds the nav-bar and all the links. Also added navbar.module.css to the styles folder. Created an Assets folder in src/assets to hold png images and logos.

Testing Considerations

Verify the nav-bar works and verify that you can implement it within other pages.

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

Screenshot 2026-09-27 at 5 55 34 PM

manavos and others added 4 commits September 24, 2026 12:34
@gitguardian

gitguardian Bot commented Sep 28, 2026

Copy link
Copy Markdown

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

Since your pull request originates from a forked repository, GitGuardian is not able to associate the secrets uncovered with secret incidents on your GitGuardian dashboard.
Skipping this check run and merging your pull request will create secret incidents on your GitGuardian dashboard.

🔎 Detected hardcoded secret in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
- - Generic Password 68a6c77 .env View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

edisontans and others added 25 commits September 28, 2026 19:32
Adds mock MenuItem data, local cart state with quantities, and a running total. Checkout not included.
Added MentuItem and mock MenuItems
feat: added contact page, no styling or submission logic
feat(order): add order page with menu list and running order summary
Removed TypeScript error suppression comment for CSS import.
feat: small about us page
This is necessary because my navbar is missing files needed..
manavos and others added 29 commits October 1, 2026 17:19
api routes created in another issue
menuItemSchema.ts already exists
Feat: Added DELETE, GET, GET ID, POST, functionality to the database.
…bar appears on every page, and contact page cleaned up
feat: replaced mockdata with api database code
feat: added input validation for frontend and server-side, including …
need to merge navbar and develop to edit navbar
feat: implemented submission completed message and reverified validation
fully added menu items on mongo and testing
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.

6 participants