[18.0][MIG] web_widget_image_download: Migration to 18.0#3046
Conversation
d0a616f to
bae4b34
Compare
bae4b34 to
ff1ce80
Compare
ff1ce80 to
3a5176e
Compare
|
Hey @OCA/web-maintainers would be great if someone could have a look at this module. |
3a5176e to
f99d5f8
Compare
|
@yajo @alessandro-fiorino @manuelregidor Could you please review this? Thanks! |
| class="o_form_binary_file_download btn btn-light border-0 rounded-circle m-1 p-1" | ||
| data-tooltip="Download" | ||
| aria-label="Download" | ||
| t-on-click="onFileDownload" |
There was a problem hiding this comment.
Probably misses a .stop to avoid propagation
On runboat, when using the button it downloaded it, but asked at the same time to upload a new image....
There was a problem hiding this comment.
Thanks for pointing this out. I added the missing .stop() to prevent the event from propagating further. I also tested it on Runboat, and the issue is now resolved—the button downloads the image without simultaneously triggering the upload action.
Currently translated at 100.0% (1 of 1 strings) Translation: web-12.0/web-12.0-web_widget_image_download Translate-URL: https://translation.odoo-community.org/projects/web-12-0/web-12-0-web_widget_image_download/zh_CN/
Currently translated at 100.0% (1 of 1 strings) Translation: web-12.0/web-12.0-web_widget_image_download Translate-URL: https://translation.odoo-community.org/projects/web-12-0/web-12-0-web_widget_image_download/pt/
Currently translated at 100.0% (1 of 1 strings) Translation: web-13.0/web-13.0-web_widget_image_download Translate-URL: https://translation.odoo-community.org/projects/web-13-0/web-13-0-web_widget_image_download/es/
Currently translated at 100.0% (1 of 1 strings) Translation: web-14.0/web-14.0-web_widget_image_download Translate-URL: https://translation.odoo-community.org/projects/web-14-0/web-14-0-web_widget_image_download/it/
Currently translated at 100.0% (1 of 1 strings) Translation: web-15.0/web-15.0-web_widget_image_download Translate-URL: https://translation.odoo-community.org/projects/web-15-0/web-15-0-web_widget_image_download/it/
f99d5f8 to
c2205fe
Compare
etobella
left a comment
There was a problem hiding this comment.
Code and Functional Review
/ocabot migration web_widget_image_download
|
/ocabot merge nobump |
|
Hey, thanks for contributing! Proceeding to merge this for you. |
|
Congratulations, your PR was merged at 7f257fe. Thanks a lot for contributing to OCA. ❤️ |
No description provided.