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

delay for search after typing #71

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sampaiodiego
Copy link
Collaborator

I've added a simple delay to make the subscribe after typing to prevent a possible overhead on server for performing a find on every character typed.

@marceloschmidt
Copy link

+1

@mizzao
Copy link
Collaborator

mizzao commented Feb 27, 2015

Happy to do some debouncing here, but I think there is already an inherent one built-in through the reactive computation (on the client).

Have you found that autocomplete is causing performance issues on your server?

@jamesgibson14
Copy link

Adding debouncing or just pulling this into the Master would be great.

@mizzao
Copy link
Collaborator

mizzao commented Jun 29, 2016

Hi @sampaiodiego - I don't have time to maintain this project anymore, so I gave you commit access to make any changes you'd like. It would be nice to follow the style used by the rest of the code and add some tests to this patch.

If you'd like permission to publish to Meteor under the existing project name, please feel free to send me your Meteor developer username.

@mizzao
Copy link
Collaborator

mizzao commented Aug 30, 2016

You have been added! https://atmospherejs.com/mizzao/autocomplete

Please do whatever you'd like to maintain and improve the project.

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.

5 participants