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

Request taxons without children from autocomplete #17

Merged
merged 1 commit into from
May 7, 2015
Merged

Request taxons without children from autocomplete #17

merged 1 commit into from
May 7, 2015

Conversation

jhawthorn
Copy link
Contributor

Without this the response is large and very slow to render. We only need
this to determine the pretty name, so this is of no use.

@jhawthorn
Copy link
Contributor Author

test this please

@@ -8,6 +8,8 @@ var set_taxon_select = function(){
initSelection: function (element, callback) {
var url = Spree.url(Spree.routes.taxons_search, {
ids: element.val(),
ids: element.val(),
Copy link
Contributor

Choose a reason for hiding this comment

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

¿qué?

Without this the response is large and very slow to render. We only need
this to determine the pretty name, so this is of no use.
@gmacdougall
Copy link
Member

👍

1 similar comment
@athal7
Copy link
Contributor

athal7 commented May 7, 2015

👍

gmacdougall added a commit that referenced this pull request May 7, 2015
Request taxons without children from autocomplete
@gmacdougall gmacdougall merged commit 0030070 into solidusio:master May 7, 2015
@jhawthorn jhawthorn deleted the taxons_without_children branch June 22, 2015 16:51
Senjai pushed a commit to Senjai/solidus that referenced this pull request Feb 17, 2016
jordan-brough added a commit to jordan-brough/solidus that referenced this pull request Aug 8, 2017
Cherry-pick current-store-selector updates from Solidus master
mamhoff added a commit to mamhoff/solidus that referenced this pull request Jun 26, 2024
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.

4 participants