-
-
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
Created new sub category in resources #2132
Conversation
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, Amanmawar17, 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! 😀
@Amanmawar17 If you don't link the issue correctly, even after the PR is merged, the issue might stay open, causing huge problems. Including it in title won't work. You can see here, if the issue is linked. I've done it for you. Just a heads-up to help you link issue properly next time. Have a great day! |
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.
@Amanmawar17
Please add the description of the subcategory.
It will be in TopBar/CategoryDescriptions.ts
I resolved this issue but can you tell me how I made a PR for new changes? |
You need to make changes in the same branch. When you push it to github in your fork, it will automatically be included in this PR. |
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.
Hi @Amanmawar17. The descriptions in your PR needs more improvement, so I provided suggestions to help you. Other than that, you're off to a great start! 😊
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.
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.
🚀 Looks good to me.
@Amanmawar17
It's fine, we will change it if it feels odd later on.
@CBID2 |
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.
Looks great
Found some conflicts that need merging @Amanmawar17 |
@Amanmawar17 |
aefc42a
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.
lgtm
HI @Amanmawar17 please resolve the merge conflicts |
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.
lgtm
@Amanmawar17 please resolve merge conflicts |
I'm closing this issue due to inactivity. Please feel free to reopen it if you'd like to contribute or have further discussions. |
Fixes Issue
closes #2107
Changes proposed
Added a new sub-category in the resources tab which was self-paced courses.
I made a new JSON file in the resources folder named self-paced courses.
I committed changes data.ts and index.ts to import new self-paced-course.json for this to be shown in the category.
Screenshots
Note to reviewers
Suggest any required changes!!
Thank You.