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
Update wagtail imports #379
Conversation
Codecov Report
@@ Coverage Diff @@
## master #379 +/- ##
==========================================
+ Coverage 98.07% 98.26% +0.19%
==========================================
Files 88 89 +1
Lines 2855 2945 +90
==========================================
+ Hits 2800 2894 +94
+ Misses 55 51 -4
Continue to review full report at Codecov.
|
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.
Thank you @danbentley. This is really helpful of you. Just spotted one small typo, but otherwise this is looking great. Thanks for the help :)
from wagtail.wagtailimages import get_image_model_string | ||
from wagtail.wagtailimages.edit_handlers import ImageChooserPanel | ||
from wagtail.wagtailadmin.edit_handlers import FieldPanel, PageChooserPanel | ||
from wagtail.iamges import get_image_model_string |
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.
Just one small typo here
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.
Thanks! 😳
I've pushed an additional commit with the typo fixed.
I've updated the documentation to match the module path changes made in Wagtail 2.0.
https://github.com/wagtail/wagtail/blob/882f8f3cf8ddd79c30e611a48882b309e90dad0c/docs/releases/2.0.rst#wagtail-module-path-updates
I left the reference to
wagtailcore
in the2.1.2
release notes since the module path was accurate at the time though happy to update that too.https://github.com/rkhleics/wagtailmenus/blob/master/docs/source/releases/2.1.2.rst
Additionally, I've fixed an issue in the hook example code. Happy to break out into a separate PR if you prefer.
Thanks!