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
[3.0] Improve import form compatibility #1434
[3.0] Improve import form compatibility #1434
Conversation
Hi @matthewhegarty, something like this would make the upgrade to 3.0 a bit easier. Please let me know what you think. |
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.
Looks good. Thanks for testing and creating this PR.
I expect you will add a note in the changelog under breaking changes as well?
@manelclos shall I write some tests to get this over the line? |
@matthewhegarty working on this today, lets see how much I can get done. |
working on the test, feel free to correct the breaking changes note I just added. |
Hi @matthewhegarty tests just added. Let me know what you think. I'll squash commits before merge. |
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.
LGTM. Presumably you were able to manually test the fix in your code base which subclassed Form (not ImportExportFormBase)
Yes, after using the new base class everything works as expected. |
c9abcc6
to
3f21c2f
Compare
squashed, ready for merge |
Fix for #1432
Avoid crashing and print a warning about what needs to be changed to get it fully working in 3.0