search

Debangan202Ok / Data-Structure

This project aims to demonstrate various data structures using Vite.js, Tailwind CSS, React. Contributions are welcome! If you have new ideas or improvements, please feel free to share them with me.
https://data-structure-skill.netlify.app/
14 stars 22 forks source link

feat: Added sidebar navigation on labs page #44

Closed sohamtembhurne closed 8 months ago

sohamtembhurne commented 8 months ago

Added navigation to sidebar on labs page by using useMemo hook.

netlify[bot] commented 8 months ago

Deploy Preview for data-structure-skill ready!

Name Link
Latest commit 944d057a175db468675ceb6dc426d3cba2d6a2c7
Latest deploy log https://app.netlify.com/sites/data-structure-skill/deploys/6538bc71851d0e00085549a2
Deploy Preview https://deploy-preview-44--data-structure-skill.netlify.app/
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Debangan202Ok commented 8 months ago

Added navigation to sidebar on labs page by using useMemo hook.

Hey @sohamtembhurne well done! It's look very nice. Currently I'm upgrading the project with nextjs. So for now I'm holding this pr. I will add this with the upgraded one. Btw nice work❤️

sohamtembhurne commented 8 months ago

Added navigation to sidebar on labs page by using useMemo hook.

Hey @sohamtembhurne well done! It's look very nice. Currently I'm upgrading the project with nextjs. So for now I'm holding this pr. I will add this with the upgraded one. Btw nice work❤️

It would be great if you could merge this under hacktoberfest. Thanks.

sohamtembhurne commented 8 months ago

Hey @Debangan202Ok, any updates on moving to NextJs ?

Debangan202Ok commented 8 months ago

Hey @Debangan202Ok, any updates on moving to NextJs ?

@sohamtembhurne actually I'm little bit busy for a week. I will give a update regarding this. Thanks for your message.

sohamtembhurne commented 8 months ago

Hey @Debangan202Ok, any updates on moving to NextJs ?

@sohamtembhurne actually I'm little bit busy for a week. I will give a update regarding this. Thanks for your message.

Any chance you could get this merged 😅 today ? Thanks.