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

select2 widget not initialized on plone5 #48

Closed

Conversation

djowett-ftw
Copy link
Contributor

resolves #42

BUT #46 is not resolved, so this shouldn't be released without a fix for that ticket

@djowett-ftw djowett-ftw requested a review from a team December 9, 2019 15:48
@djowett-ftw djowett-ftw self-assigned this Dec 10, 2019
Copy link
Member

@jone jone left a comment

Choose a reason for hiding this comment

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

Does this PR contain all the necessary changes?
Where does hackS2 come from?

I think I don't get why this fixes the problem of the references issue.
Please also add a changelog entry 😉

@djowett-ftw djowett-ftw force-pushed the dj_select2_widget_not_initialized_on_plone5 branch from 8bae78b to 22c7863 Compare December 12, 2019 15:10
@djowett-ftw
Copy link
Contributor Author

djowett-ftw commented Dec 12, 2019

Thanks for looking

Does this PR contain all the necessary changes?

Yes - for this issue

Where does hackS2 come from?

It is in the JS above - inserted previously to prevent conflict between Plone's v3.5 select2 and v4.0.2 in this product

I think I don't get why this fixes the problem of the references issue.

I think you're thinking of the fix in #46?

Please also add a changelog entry 😉

Done - good call :-)

@djowett-ftw
Copy link
Contributor Author

This fix is included in #47 which has just been merged, so closing this

@djowett-ftw djowett-ftw deleted the dj_select2_widget_not_initialized_on_plone5 branch December 13, 2019 08:16
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.

Select2 widget not initialised when not on default tab plone 5
2 participants