-
-
Notifications
You must be signed in to change notification settings - Fork 161
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
[14.0][IMP] storage_image: Add a wizard to replace related file #273
[14.0][IMP] storage_image: Add a wizard to replace related file #273
Conversation
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 for the improvement :)
Can you
- add the wizard to storage_media too
- drop the onchange approach
- split commit by module
- drop the funny image
- fix pre-commit
please? 😄
d41ea04
to
c0a397e
Compare
baaah, you're not funny |
6116a51
to
501fc5e
Compare
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.
LG, pls drop the icon. It's automatically generated ;)
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.
Some tests
directory are missing an __init__.py
file
501fc5e
to
74456d8
Compare
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
@TDu could you update your review pls? |
/ocabot merge minor |
On my way to merge this fine PR! |
Congratulations, your PR was merged at 5b6492c. Thanks a lot for contributing to OCA. ❤️ |
Adds a wizard on the image form to replace the attached file.