-
Notifications
You must be signed in to change notification settings - Fork 86
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
Drop pre Wagtail 2.15 support #617
Conversation
5578fa3
to
9c1d573
Compare
Codecov ReportBase: 91.06% // Head: 91.80% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #617 +/- ##
==========================================
+ Coverage 91.06% 91.80% +0.74%
==========================================
Files 49 46 -3
Lines 3971 3916 -55
Branches 606 593 -13
==========================================
- Hits 3616 3595 -21
+ Misses 209 188 -21
+ Partials 146 133 -13
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
<script src="{% versioned_static 'wagtaildocs/js/document-chooser-modal.js' %}"></script> | ||
<script src="{% versioned_static 'wagtaildocs/js/document-chooser.js' %}"></script> | ||
<script src="{% versioned_static 'wagtailadmin/js/chooser-modal.js' %}"></script> | ||
<script src="{% versioned_static 'wagtailsnippets/js/snippet-chooser.js' %}"></script> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
considering we redirect users to edit choosers, rather than allow them to choose another one I am not sure we need these, but that's for another PR
9c1d573
to
8e57b1b
Compare
The previous change was needed because of the Django version check see 39c7da7
943bce3
to
21d5d1c
Compare
edit_translation.html
(Editing translation fails with 1.3a2: snippet-chooser-modal.js no longer exists in wagtail #618)