Skip to content

[Autocomplete] add option to specify minimum characters for autocomplete #492

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

Merged

Conversation

daFish
Copy link
Contributor

@daFish daFish commented Oct 7, 2022

Q A
Bug fix? no
New feature? yes
Tickets Fix #432
License MIT

This change adds a new option min_characters for specifying the value of entered characters before the autocomplete is triggerd. This avoids calls with an empty query.

Todos:

  • add tests

@weaverryan I also added a CHANGELOG.md because this file wasn't present.

@daFish daFish force-pushed the feat/gh-432-autocomplete-minimum-characters branch from 3b76cae to adc5176 Compare October 7, 2022 12:27
@daFish daFish marked this pull request as ready for review October 7, 2022 12:41
@daFish daFish changed the title feat(autocomplete): add option to specify minimum characters for autocomplete [Autocomplete] add option to specify minimum characters for autocomplete Oct 7, 2022
Copy link
Member

@weaverryan weaverryan left a comment

Choose a reason for hiding this comment

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

Some notes - but looking good!

@daFish daFish force-pushed the feat/gh-432-autocomplete-minimum-characters branch from adc5176 to e3b6905 Compare October 7, 2022 19:40
@daFish
Copy link
Contributor Author

daFish commented Oct 7, 2022

@weaverryan Should all be fine now and all checks are happy.

@daFish daFish force-pushed the feat/gh-432-autocomplete-minimum-characters branch from e3b6905 to 4c55dcb Compare October 7, 2022 19:44
@weaverryan weaverryan force-pushed the feat/gh-432-autocomplete-minimum-characters branch from 4c55dcb to 604ad69 Compare October 12, 2022 13:46
@weaverryan
Copy link
Member

Thank you Marcus!

@weaverryan weaverryan merged commit 11fae63 into symfony:2.x Oct 12, 2022
@daFish daFish deleted the feat/gh-432-autocomplete-minimum-characters branch October 12, 2022 14:39
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.

[Autocomplete] Add a way to require a minimum char number to before send ajax request
2 participants