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

Create nav component #158

Merged
merged 18 commits into from
Apr 30, 2024
Merged

Create nav component #158

merged 18 commits into from
Apr 30, 2024

Conversation

ryandotfurrer
Copy link
Member

@ryandotfurrer ryandotfurrer commented Apr 25, 2024

Closes #155

Created nav component

Screenshot of Nav component in dark mode

CleanShot.2024-04-26Arc000748.mp4

Todo

  • Add hamburger icon
    - [ ] Add logic to show sign out only if user is signed in (?) Not needed for MVP
  • Integrate Button.tsx component from Created custom button variants #132 and use in place of standard <button> element

Copy link

appwrite bot commented Apr 25, 2024

Gridiron Survivor Application 6616ea581ef9f5521c7d

Function ID Status Action
Your function deployment has failed. Please check the logs for more details and retry.

Project name: Gridiron Survivor Application
Project ID: 6616ea581ef9f5521c7d

Function ID Status Action
userAuth 6626fef885a9f630442b failed Failed View Logs

Only deployments on the production branch are activated automatically. If you'd like to activate this deployment, navigate to your deployments. Learn more about Appwrite Function deployments.

💡 Did you know?
Appwrite has a Discord community with over 16 000 members. Come join us!

Copy link

vercel bot commented Apr 25, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
gridiron-survivor ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 30, 2024 0:49am
gridiron-survivor-storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 30, 2024 0:49am

…AuthButton from page.tsx, and remove Nav from layout.tsx
@ryandotfurrer ryandotfurrer merged commit fff3d94 into develop Apr 30, 2024
7 checks passed
@ryandotfurrer ryandotfurrer deleted the rf_create-nav-component branch April 30, 2024 01:08
@ryandotfurrer
Copy link
Member Author

PR approved. Squashing and merging.

Clue355 pushed a commit that referenced this pull request Apr 30, 2024
Closes #155 

# Created nav component
![Screenshot of Nav component in dark
mode](https://github.com/LetsGetTechnical/gridiron-survivor/assets/40150036/954db134-5f79-47d3-aa6f-df152bf9b4b8)


https://github.com/LetsGetTechnical/gridiron-survivor/assets/40150036/0e143fbb-a8f7-45b9-9465-0983c3dfbc10

## Todo
- [x] Add hamburger icon
~~- [ ] Add logic to show sign out only if user is signed in (?)~~ Not
needed for MVP
- [x] Integrate `Button.tsx` component from #132 and use in place of
standard `<button>` element

---------

Co-authored-by: Shashi Lo <[email protected]>
Clue355 pushed a commit that referenced this pull request May 1, 2024
Closes #155 

# Created nav component
![Screenshot of Nav component in dark
mode](https://github.com/LetsGetTechnical/gridiron-survivor/assets/40150036/954db134-5f79-47d3-aa6f-df152bf9b4b8)


https://github.com/LetsGetTechnical/gridiron-survivor/assets/40150036/0e143fbb-a8f7-45b9-9465-0983c3dfbc10

## Todo
- [x] Add hamburger icon
~~- [ ] Add logic to show sign out only if user is signed in (?)~~ Not
needed for MVP
- [x] Integrate `Button.tsx` component from #132 and use in place of
standard `<button>` element

---------

Co-authored-by: Shashi Lo <[email protected]>
Clue355 pushed a commit that referenced this pull request May 1, 2024
Closes #155

![Screenshot of Nav component in dark
mode](https://github.com/LetsGetTechnical/gridiron-survivor/assets/40150036/954db134-5f79-47d3-aa6f-df152bf9b4b8)

https://github.com/LetsGetTechnical/gridiron-survivor/assets/40150036/0e143fbb-a8f7-45b9-9465-0983c3dfbc10

- [x] Add hamburger icon
~~- [ ] Add logic to show sign out only if user is signed in (?)~~ Not
needed for MVP
- [x] Integrate `Button.tsx` component from #132 and use in place of
standard `<button>` element

---------

Co-authored-by: Shashi Lo <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

Create nav component
4 participants