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 jquery-ui autocomplete. #514

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

Conversation

bahudso
Copy link

@bahudso bahudso commented Apr 2, 2020

Resolves issue #510 by changing the autocomplete path in backend.js

@bricesanchez
Copy link
Member

Hi @bahudso,

Thanks for your pull request!

Could give us a way to verify if this change fix the issue?

When i see the official repo of jquery-ui, it looks like we needs the widgets folder : https://github.com/jquery-ui-rails/jquery-ui-rails/search?q=autocomplete&unscoped_q=autocomplete

@bahudso
Copy link
Author

bahudso commented Apr 6, 2020

@bricesanchez I was able to reproduce the same issue from #510 on multiple projects. Overriding the require path, per that PR, was the only thing that resolved the issue for me.

There doesn't seem to be much discussion around #510 so I'm not sure if it's been investigated yet.

@parndt
Copy link
Member

parndt commented Aug 11, 2020

@bricesanchez @bahudso this was implemented into 4-0-stable by #517

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.

None yet

3 participants