Skip to content

Conversation

ellinge
Copy link
Collaborator

@ellinge ellinge commented Jun 27, 2019

What does it do?

Adds an additional property, searchPredicate, which is used when filtering/searching for nodes.
Can be used to have acronyms and other metadata, which should cause a match/hit, or to implement a fuzzy search.

Fixes #269

Type of change

  • New feature

@ellinge ellinge changed the title fix: Add support for additional filtering text fix: Add support for additional filtering text on nodes Jun 27, 2019
@coveralls
Copy link

coveralls commented Jun 27, 2019

Pull Request Test Coverage Report for Build 1249

  • 8 of 8 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.03%) to 95.194%

Totals Coverage Status
Change from base Build 1234: 0.03%
Covered Lines: 574
Relevant Lines: 587

💛 - Coveralls

@ellinge ellinge changed the title fix: Add support for additional filtering text on nodes fix: Add support for custom search filtering on nodes Jun 27, 2019
Copy link
Collaborator

@mrchief mrchief 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 barring minor tweaks

@mrchief mrchief changed the title fix: Add support for custom search filtering on nodes feat: Add support for custom search filtering on nodes Jun 28, 2019
@qlty-cloud-legacy
Copy link

Code Climate has analyzed commit 1a52123 and detected 0 issues on this pull request.

View more on Code Climate.

Copy link
Collaborator

@mrchief mrchief left a comment

Choose a reason for hiding this comment

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

Yet another useful addition!! 👏

@mrchief mrchief changed the title feat: Add support for custom search filtering on nodes feat: Add support for custom search filtering on nodes (#270) ✨ Jun 29, 2019
@mrchief mrchief merged commit 9cffc90 into develop Jun 29, 2019
@mrchief mrchief deleted the fix/#269 branch June 29, 2019 14:29
- [always](#always)
- [form states (disabled|readOnly)](#formstates)
- [id](#id)
- [searchPredicate](#searchPredicate)
Copy link
Collaborator

Choose a reason for hiding this comment

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

This needs to be all lowercase like (#searchpredicate). GH doesn't link otherwise. Gets me everytime. :)

But don't submit another PR. I'll be fixing this in a doc update. Experimenting with a new readme layout that doesn't need so much scrolling.

Something like

image

mrchief pushed a commit that referenced this pull request Jul 23, 2019
New prop: `searchPredicate` - Adds support to filter nodes using custom search logic. For details, checkout README.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add possibility to use custom predicate/filter when searching nodes
3 participants