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

Fix race-condition issue with ajax sources #28

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mneuhaus
Copy link

currently, if you use a ajax call in your source callback you can run into issues
where results are stored under a wrong search term. This change adds an additional
argument to the source callback to specify the search term in order to gurantee
the storage of the results under the correct search term.

currently, if you use a ajax call in your
source callback you can run into issues
where results are stored under a wrong
search term. This change adds an additional
argument to the source callback to specify
the search term in order to gurantee
the storage of the results under the
correct search term
@tremby
Copy link

tremby commented Sep 30, 2016

I had the same issue.

@tremby tremby mentioned this pull request Sep 30, 2016
@taitems
Copy link

taitems commented Jan 26, 2018

Can confirm this fixes my issue as well, and should be merged.

This was referenced Oct 6, 2023
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.

3 participants