-
Notifications
You must be signed in to change notification settings - Fork 151
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
Solved the issue #316 Added My canteen button in navbar #333
Conversation
@Praveenkumar5048 is attempting to deploy a commit to the vansh69's projects Team on Vercel. A member of the Team first needs to authorize it. |
@hustlerZzZ hlo project admin, while doing the above task i have gone through entire code of this repository. During this i have encountered many bug in the code specially related to autentication, and token.
I would like to work on this to solve this bug. |
Ok sure |
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.
Great work
Congratulations, Your pull request has been successfully merged 🥳🎉 Thank you for your contribution to the project 🚀 Keep Contributing!! ✨ |
I haved added a separate button called "My Canteen" in the navbar which is visible to only those who logged in as canteen user. Through this button they can access the details of the canteen that is it redirects them to section/:canteenId route.
I implemented this using conditional rendering. The code was already storing the canteenId in the local storage when user signUp to the application. In my code i am checking whether the canteenId exists in local Storage to conditionally render "My canteen" button.
Fixes #316
Type of change
Please mark with an X the type that applies:
I have tested my code, it works perfectly without any errors.
Screenshots and Videos
Checklist
Please mark with an X the items that apply: