-
-
Notifications
You must be signed in to change notification settings - Fork 545
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
Feat: add new design for categories page #2195
Conversation
* feat: Add Swift Sub Category * Fix small correction * Update database/languages/swift.json Co-authored-by: Christine Belzie <[email protected]> --------- Co-authored-by: Christine Belzie <[email protected]>
* [ADDED]: BlockChain Category * Changes done * Update index.ts * Update index.ts * Changes made * Update CategoryDescriptions.ts
* feat: add iconsax icon * chore: update description Co-authored-by: Rupali Haldiya <[email protected]> --------- Co-authored-by: Anmol Baranwal <[email protected]> Co-authored-by: Rupali Haldiya <[email protected]>
* Added 2 new podcasts * Added Overpowered.AI podcast * Added new link to podcasts * fix: json structure * chore: update description Co-authored-by: Christine Belzie <[email protected]> * chore: remove whitespace Co-authored-by: Aftab Rehan <[email protected]> --------- Co-authored-by: Anmol Baranwal <[email protected]> Co-authored-by: Christine Belzie <[email protected]> Co-authored-by: Aftab Rehan <[email protected]>
* UI libraries in fronted added * added description for the UI libraries * description update for materailui , antdesign, chakraui * removed os tutorials
* Added new course * Added new document
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.
Thank you, gakshita, for creating this pull request and contributing to LinksHub! 💗
The maintainers will review this Pull Request and provide feedback as soon as possible! 😇
We appreciate your patience and contribution, Keep up the great work! 😀
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.
The page looks good @gakshita, but can you revert the commits that don't pertain to the issue that your PR solves?
Have removed the unecessary commits |
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.
Excellent PR, @gakshita 💪 🚀 💯
Thanks for the contribution. It just needs minor adjustments and then we're good to go.
However, I have only reviewed the code yet. I'll proceed with UI testing and provide my feedback in a while. Feel free to ask if you have any questions or want clarification on something.
No need to stress! If you ever need assistance, feel free to reach out. We're here to support and help you on your learning journey. 🚀 |
@aftabrehan @rupali-codes @CBID2 all changes have been fixed from my side! |
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.
Thanks @gakshita! The code looks much better now.
However, here are a few suggestions from my side.
It all looks super awesome 🙌. Great work @gakshita. Please, help to adjust these two things as well, and then we're good to merge this PR.
|
@gakshita |
Couldn't find item description data i the object, rest changes have been pushed. |
@gakshita, Please, check this for reference. LinksHub/components/TopBar/TopBar.tsx Lines 17 to 28 in eba4b81
|
Any updates @gakshita? 😁 |
@gakshita you still working on it? |
Someone is attempting to deploy a commit to a Personal Account owned by @rupali-codes on Vercel. @rupali-codes first needs to authorize it. |
fixes #2177
Changes proposed
New design for category page implemented