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

Fixes re-initialization upon adding inlines #157

Merged
merged 1 commit into from
Feb 3, 2016
Merged

Fixes re-initialization upon adding inlines #157

merged 1 commit into from
Feb 3, 2016

Conversation

funkyfuture
Copy link
Contributor

the rationale is that the elements an event are bound to must exist. this isn't the case for the specified selectors.
i'd appreciate if this was released soon.

solves #87

the rationale is that the elements an event are bound to must exist. this isn't the case for the specified selectors.
i'd appreciate if this was released soon.

solves #87
@funkyfuture
Copy link
Contributor Author

the failed tests seem to relate to Py3.2. i'm not sure if it is still supported anyway.

@crucialfelix
Copy link
Owner

Hm. It seems to be an error in pip for 3.2

py32-dj15 create: /home/travis/build/crucialfelix/django-ajax-selects/.tox/py32-dj15
ERROR: invocation failed (exit code 1), logfile: /home/travis/build/crucialfelix/django-ajax-selects/.tox/py32-dj15/log/py32-dj15-0.log
ERROR: actionid: py32-dj15

I can't find anything about Django not supporting 3.2. Have they dropped that ?

@crucialfelix
Copy link
Owner

The PR looks good. That's the correct solution I think. Thanks

crucialfelix added a commit that referenced this pull request Feb 3, 2016
Fixes re-initialization upon adding inlines
@crucialfelix crucialfelix merged commit 6aaddd1 into crucialfelix:develop Feb 3, 2016
@funkyfuture
Copy link
Contributor Author

i don't know exactly the status of Py3.2, just noticed that it often fails in testsuites or isn't tested at all.

can you anticipate when this bug fix will be available via the PyPI?

@crucialfelix
Copy link
Owner

I'm not quite sure yet. Very busy at the moment (2 clients and a new born baby). If you need to deploy right away you can always use a github link with pip.

@funkyfuture
Copy link
Contributor Author

a'ight, all the best with the new one! sure, we can use a link to the repo. and keep in mind to change it later. ;-)

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

2 participants