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

feat: enableClear option for SearchSelect component #681

Merged

Conversation

sudhamjayanthi
Copy link
Contributor

@sudhamjayanthi sudhamjayanthi commented Sep 16, 2023

Description
Implemented the enableClear functionality in SearchSelect similar to Search component.

Related issue(s)
#680 (comment)

What kind of change does this PR introduce? (check at least one)

  • Bug fix (non-breaking change which fixes an issue)
  • New Feature (non-breaking change which adds functionality)
  • Refactor
  • Build-related changes
  • Other, please describe:

Does this PR introduce a breaking change? (check one)

  • Yes
  • No

Screenshots (if appropriate):
demo gif

The PR fulfills these requirements:

  • It's submitted to the main branch
  • When resolving a specific issue, it's referenced in the related issue section above
  • My change requires a change to the documentation. (Managed by Tremor Team)
  • Check the "Allow edits from maintainers" option while creating your PR.
  • Add refs #XXX or fixes #XXX to the related issue section if your PR refers to or fixes an issue.
  • I have added tests to cover my changes

PS: I needed this for my project, but thought it would be better to contribute for everyone else like me. Feel free to let me know if any changes are required :) Otherwise kindly merge it!

Signed-off-by: Sudham Jayanthi <workwithsudham@gmail.com>
@sudhamjayanthi sudhamjayanthi marked this pull request as ready for review September 16, 2023 16:50
@sudhamjayanthi
Copy link
Contributor Author

thanks for the changes @severinlandolt! can we merge this?

@severinlandolt
Copy link
Member

Yes, will be included in the next minor version.

@severinlandolt severinlandolt changed the base branch from main to beta September 25, 2023 15:21
@severinlandolt severinlandolt self-assigned this Sep 25, 2023
@severinlandolt severinlandolt added the PR: In Review This PR is in the process of being reviewed by the team label Sep 25, 2023
@severinlandolt severinlandolt changed the base branch from beta to main September 26, 2023 11:58
@severinlandolt
Copy link
Member

Sorry, will make it into v3.9

@sudhamjayanthi
Copy link
Contributor Author

hey @severinlandolt, do we have a estimated release date for v3.9?

@severinlandolt
Copy link
Member

ca. mid October

@severinlandolt severinlandolt changed the base branch from main to beta September 30, 2023 17:02
@severinlandolt severinlandolt merged commit 0d21430 into tremorlabs:beta Sep 30, 2023
12 checks passed
@github-actions
Copy link

🎉 This PR is included in version 3.9.0-beta.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: In Review This PR is in the process of being reviewed by the team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants