Skip to content

feat: allow to pass a getFn for a specific key#627

Merged
krisk merged 2 commits intokrisk:masterfrom
fakenickels:master
May 3, 2022
Merged

feat: allow to pass a getFn for a specific key#627
krisk merged 2 commits intokrisk:masterfrom
fakenickels:master

Conversation

@fakenickels
Copy link
Copy Markdown
Contributor

@fakenickels fakenickels commented Feb 23, 2022 •
edited
Loading

This PR introduces the following

const fuse = new Fuse(Books, {
  useExtendedSearch: true,
  includeMatches: true,
  includeScore: true,
  threshold: 0.3,
  keys: [
    { name: "title", getFn: (book) => book.title },
    { name: "authorName", getFn: (book) => book.author.firstName },
  ],
});
const result = fuse.search({ title: "old man" });

I noticed it has been requested several times

Let me know if any other changes are necessary before merging

@fakenickels fakenickels reopened this Feb 23, 2022
@krisk krisk changed the title allow to pass a getFn for a specific key feat: allow to pass a getFn for a specific key Mar 2, 2022
@Aaronius
Copy link
Copy Markdown

Aaronius commented Apr 13, 2022

Yes please!

@krisk krisk merged commit 203871f into krisk:master May 3, 2022
@krisk
Copy link
Copy Markdown
Owner

krisk commented May 3, 2022 •
edited
Loading

This is great. As a follow-up to this, could you create a PR with the relevant docs updates?

@krisk krisk mentioned this pull request May 3, 2022
Merged
@krisk
Copy link
Copy Markdown
Owner

krisk commented May 3, 2022 •
edited
Loading

Could you make the requisite TS changes?

krisk pushed a commit that referenced this pull request May 3, 2022
@fakenickels @krisk
feat: allow passing getFn for a specific key
1d445b9 
Closes #627
@usernamerandom11 usernamerandom11 mentioned this pull request Sep 7, 2024
Open
@usernamerandom11 usernamerandom11 mentioned this pull request Sep 21, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@fakenickels @Aaronius @krisk