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(entity-editor): search for duplicates when pre-filling name #801

Merged
merged 5 commits into from
Mar 14, 2022

Conversation

MonkeyDo
Copy link
Member

@MonkeyDo MonkeyDo commented Mar 10, 2022

While working on #800 I realized we have the same issue with the duplicate search:
When creating a new entity, the search for possible duplicates is only triggered when when user types in the name input.
When using query parameters to pre-fill the name of a new entity, that doesn't happen, leading to possible dupes.

With this PR we trigger a search for possible duplicates when we render the props to send to the user.

When using query parameters to pre-fill the name of a new entity, we trigger a search for possible duplicates, as we do when user types the name.
These become Lists in Redux and need to be converted accordingly at the other end.
@MonkeyDo MonkeyDo force-pushed the search-initial-pre-filled-name branch from bffa5bf to ff8c970 Compare March 14, 2022 19:40
@MonkeyDo MonkeyDo changed the title feat(entity-editor): search for possible duplicates when pre-filling name feat(entity-editor): search for duplicates when pre-filling name Mar 14, 2022
@MonkeyDo MonkeyDo merged commit 1d99bf2 into master Mar 14, 2022
@MonkeyDo MonkeyDo deleted the search-initial-pre-filled-name branch March 14, 2022 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant