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

fix: clear button disable when there is no filter applied #1654 #1655

Merged
merged 2 commits into from
Jul 9, 2023

Conversation

biplavmz
Copy link
Contributor

@biplavmz biplavmz commented Jul 9, 2023

Related Issue

Description

clear button disable when there is no filter applied bug no #1654

Screenshots

@vercel
Copy link

vercel bot commented Jul 9, 2023

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

Name Status Preview Updated (UTC)
projectshut ✅ Ready (Inspect) Visit Preview Jul 9, 2023 3:41pm

@github-actions github-actions bot added the wait for reviewers wait for the maintainer label Jul 9, 2023
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job, @biplavmz! 🎉 Thank you for submitting your first pull request. Your contribution is valuable and we appreciate your efforts to improve our project.

We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines

@priyankarpal priyankarpal added no-issue-activity no-issue-activity changes-required changes-required no issue number contributor haven't added the issue number and removed no-issue-activity no-issue-activity labels Jul 9, 2023
Copy link
Owner

@priyankarpal priyankarpal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • add the issue number to the PR description
  • add a clear & valid PR title like this fix: clear button disable when there is no filter applied #issuenumber (replace to the issue number)
  • DON'T create PR first. If you do in future your PR might be closed.
  • Follow Conventional Commits. https://www.conventionalcommits.org/en/v1.0.0/

@biplavmz
Copy link
Contributor Author

biplavmz commented Jul 9, 2023

ok @priyankarpal thanks for clarifying the thing.

@biplavmz biplavmz changed the title app\projects\page.tsx: bug fix-1654 fix: clear button disable when there is no filter applied #1654 Jul 9, 2023
@biplavmz biplavmz changed the title fix: clear button disable when there is no filter applied #1654 fix: clear button disable when there is no filter applied bug no:- 1654 Jul 9, 2023
@biplavmz biplavmz changed the title fix: clear button disable when there is no filter applied bug no:- 1654 fix: clear button disable when there is no filter applied #1654 Jul 9, 2023
@priyankarpal priyankarpal added gssoc23 only for Girlscript summer of code level1 small level fix for GSSoC and removed no issue number contributor haven't added the issue number changes-required changes-required labels Jul 9, 2023
Copy link
Owner

@priyankarpal priyankarpal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks Good 🎉

@priyankarpal priyankarpal merged commit d86ee06 into priyankarpal:main Jul 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gssoc23 only for Girlscript summer of code level1 small level fix for GSSoC wait for reviewers wait for the maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: clear button should be disable when there is no filter tags applied or search input is empty
2 participants