Skip to content

feat: add ability to remove individual recent searches#789

Open
Anuja-3248 wants to merge 1 commit into
JhaSourav07:mainfrom
Anuja-3248:feat/individual-recent-searches
Open

feat: add ability to remove individual recent searches#789
Anuja-3248 wants to merge 1 commit into
JhaSourav07:mainfrom
Anuja-3248:feat/individual-recent-searches

Conversation

@Anuja-3248
Copy link
Copy Markdown

Closes #783
Added the ability to remove individual recent searches from the history to improve user experience.

What Changed

  • Added a button to each recent search pill in the UI.
  • Implemented removeSearch logic in the useRecentSearches hook to filter out specific usernames from local storage.
  • Retained the global "Clear" button for bulk deletion.

Why

Previously, the only way a user could manage their history was by clicking the global "Clear" button, which wiped the entire list of recent searches.
This enhancement allows users to manage their recent searches more granularly, improving the usability of the dashboard preview tool by ensuring they don't lose all of their saved history just to fix a single typo.

Pillar

  • 🎨 Pillar 1 — New Theme Design
  • 📐 Pillar 2 — Geometric SVG Improvement
  • 🕐 Pillar 3 — Timezone Logic Optimization
  • 🛠️ Other (Bug fix, refactoring, docs)

Checklist before requesting a review:

  • I have read the CONTRIBUTING.md file.
  • I have tested these changes locally (localhost:3000/api/streak?user=YOUR_USERNAME).
  • I have run npm run lint locally and resolved all errors.
  • I have run the relevant Vitest test suite successfully.
  • My commits follow the Conventional Commits format (e.g., feat(themes): ..., fix(calculate): ...).
  • I have updated README.md if I added a new theme or URL parameter.
  • I have starred the repo.
  • I have made sure that I have only one commit to merge in this PR.
  • The SVG output matches the CommitPulse "premium quality" aesthetic standard.
  • (Recommended) I joined the CommitPulse Discord community for contributor discussions, mentorship, and faster PR support.
    12:00 AM

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 27, 2026

@Anuja-3248 is attempting to deploy a commit to the jhasourav07's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link
Copy Markdown

👋 Hey @Anuja-3248, welcome to CommitPulse! 🎉

Thanks for opening your first pull request — this is a big deal and we appreciate the effort!

While you wait for a review, please double-check:

  • ✅ You've read the CONTRIBUTING.md checklist
  • npm run lint, npm run format, and npm run test all pass locally
  • ✅ Your PR has a visual preview if it touches any SVG output
  • 💬 You've joined our Discord for faster PR feedback

A maintainer will review your PR shortly. Hang tight! 🚀

@github-actions github-actions Bot added the needs-details This PR is missing required description details. label May 27, 2026
@github-actions
Copy link
Copy Markdown

👋 Hey @Anuja-3248, it looks like you didn't use our PR template!

The section ## Description is missing from your PR description.

Please update your PR description to include all required sections so we can review this properly:

  • ## Description — What does this PR do? Which issue does it fix?
  • ## Pillar — Which contribution pillar does this fall under?
  • ## Checklist — Have you ticked off the quality checklist?

You can find the full template in CONTRIBUTING.md. Just edit your PR description and the needs-details label will be removed automatically. 🙌

@github-actions github-actions Bot added type:feature New features, additions, or enhancements needs-rebase This PR has merge conflicts and needs a rebase. labels May 27, 2026
@github-actions
Copy link
Copy Markdown

⚠️ Hey @Anuja-3248, this PR has merge conflicts with the main branch.

Please pull the latest changes and resolve the conflicts so we can review it!

git fetch origin
git rebase origin/main
# resolve any conflicts, then:
git push --force-with-lease

Once resolved, the needs-rebase label will be removed automatically on the next check. 🙌

1 similar comment
@github-actions
Copy link
Copy Markdown

⚠️ Hey @Anuja-3248, this PR has merge conflicts with the main branch.

Please pull the latest changes and resolve the conflicts so we can review it!

git fetch origin
git rebase origin/main
# resolve any conflicts, then:
git push --force-with-lease

Once resolved, the needs-rebase label will be removed automatically on the next check. 🙌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-details This PR is missing required description details. needs-rebase This PR has merge conflicts and needs a rebase. type:feature New features, additions, or enhancements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: add ability to remove individual items from "Recent Searches"

1 participant