Skip to content

[18.0][MIG] web_copy_confirm: Migration to 18.0#3242

Merged
OCA-git-bot merged 20 commits into
OCA:18.0from
ForgeFlow:18.0-mig-web_copy_confirm
Aug 18, 2025
Merged

[18.0][MIG] web_copy_confirm: Migration to 18.0#3242
OCA-git-bot merged 20 commits into
OCA:18.0from
ForgeFlow:18.0-mig-web_copy_confirm

Conversation

@ArnauCForgeFlow

@ArnauCForgeFlow ArnauCForgeFlow commented Aug 6, 2025

Copy link
Copy Markdown
Contributor

Standard Migration to v18.0

StefanRijnhart and others added 19 commits August 6, 2025 15:52
@StefanRijnhart @ArnauCForgeFlow
[ADD] Ask for confirmation when copying records
c31a4d1 
[UPD] Update web_copy_confirm.pot

[ADD] icon.png
@rconjour @ArnauCForgeFlow
[IMP] web_copy_confirm: black, isort, prettier
d0ece0a
@rconjour @ArnauCForgeFlow
[MIG] web_copy_confirm: Migration to 14.0
0b835b8
@oca-travis @ArnauCForgeFlow
[UPD] Update web_copy_confirm.pot
67dd578
@OCA-git-bot @ArnauCForgeFlow
[UPD] README.rst
9ca818b
@Corneliuus @ArnauCForgeFlow
Added translation using Weblate (German)
422a8cb
@Corneliuus @ArnauCForgeFlow
Translated using Weblate (German)
3df5287 
Currently translated at 100.0% (2 of 2 strings)

Translation: web-14.0/web-14.0-web_copy_confirm
Translate-URL: https://translation.odoo-community.org/projects/web-14-0/web-14-0-web_copy_confirm/de/
@MiquelRForgeFlow @ArnauCForgeFlow
[MIG] web_copy_confirm: Migration to 15.0
9602bc7
@OCA-git-bot @ArnauCForgeFlow
[UPD] README.rst
043452f
@dsolanki-initos @ArnauCForgeFlow
[IMP] web_copy_confirm: pre-commit stuff
a3253de
@dsolanki-initos @ArnauCForgeFlow
[MIG] web_copy_confirm: Migration to 16.0
62929ba
@ArnauCForgeFlow
[UPD] Update web_copy_confirm.pot
b7e29c4
@OCA-git-bot @ArnauCForgeFlow
[BOT] post-merge updates
18d2cc8
@weblate @ArnauCForgeFlow
Update translation files
ed5d66b 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: web-16.0/web-16.0-web_copy_confirm
Translate-URL: https://translation.odoo-community.org/projects/web-16-0/web-16-0-web_copy_confirm/
@mymage @ArnauCForgeFlow
Added translation using Weblate (Italian)
59bf76a
@mymage @ArnauCForgeFlow
Translated using Weblate (Italian)
b8c90a8 
Currently translated at 100.0% (2 of 2 strings)

Translation: web-16.0/web-16.0-web_copy_confirm
Translate-URL: https://translation.odoo-community.org/projects/web-16-0/web-16-0-web_copy_confirm/it/
@arnaudlayec @ArnauCForgeFlow
Added translation using Weblate (French)
69f5c84
@arnaudlayec @ArnauCForgeFlow
Translated using Weblate (French)
a89a2bf 
Currently translated at 100.0% (2 of 2 strings)

Translation: web-16.0/web-16.0-web_copy_confirm
Translate-URL: https://translation.odoo-community.org/projects/web-16-0/web-16-0-web_copy_confirm/fr/
@ArnauCForgeFlow
[IMP] web_copy_confirm: pre-commit auto fixes
0934526
@ArnauCForgeFlow ArnauCForgeFlow force-pushed the 18.0-mig-web_copy_confirm branch from 45f3f74 to 7af483e Compare August 7, 2025 09:11
LoisRForgeFlow
LoisRForgeFlow reviewed Aug 7, 2025
View reviewed changes

@LoisRForgeFlow LoisRForgeFlow left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I works good from form view. However, now it is also possible to duplicate from the list view:
image

I think it should be covered too.

@ArnauCForgeFlow ArnauCForgeFlow force-pushed the 18.0-mig-web_copy_confirm branch from 7af483e to f3d3858 Compare August 7, 2025 14:30
@ArnauCForgeFlow

ArnauCForgeFlow commented Aug 7, 2025

Copy link
Copy Markdown
Contributor Author

Done, now the pop-up appears aswell in the list view.

LoisRForgeFlow
LoisRForgeFlow approved these changes Aug 12, 2025
View reviewed changes

@LoisRForgeFlow LoisRForgeFlow left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! Tested and works as expected 👍

BhaveshHeliconia
BhaveshHeliconia approved these changes Aug 14, 2025
View reviewed changes

@BhaveshHeliconia BhaveshHeliconia left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Functional review LGTM!

@OCA-git-bot

OCA-git-bot commented Aug 14, 2025

Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@LoisRForgeFlow

LoisRForgeFlow commented Aug 18, 2025

Copy link
Copy Markdown
Contributor

@OCA/web-maintainers could you merge?

CarlosRoca13
CarlosRoca13 reviewed Aug 18, 2025
View reviewed changes

@CarlosRoca13 CarlosRoca13 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just 2 comments

Comment thread web_copy_confirm/static/tests/confirm.test.js
Comment thread web_copy_confirm/static/src/js/web_copy_confirm.esm.js Outdated
@ArnauCForgeFlow ArnauCForgeFlow force-pushed the 18.0-mig-web_copy_confirm branch from f3d3858 to 0da880a Compare August 18, 2025 08:48
CarlosRoca13
CarlosRoca13 approved these changes Aug 18, 2025
View reviewed changes

@CarlosRoca13 CarlosRoca13 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/ocabot migration web_copy_confirm
/ocabot merge nobump

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Aug 18, 2025
@OCA-git-bot

OCA-git-bot commented Aug 18, 2025

Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-3242-by-CarlosRoca13-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot mentioned this pull request Aug 18, 2025
Open
49 tasks
@OCA-git-bot

OCA-git-bot commented Aug 18, 2025

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 3669255. Thanks a lot for contributing to OCA. ❤️

@OCA-git-bot OCA-git-bot merged commit 944a6c7 into OCA:18.0 Aug 18, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@CarlosRoca13 CarlosRoca13 CarlosRoca13 approved these changes
+2 more reviewers
@LoisRForgeFlow LoisRForgeFlow LoisRForgeFlow approved these changes
@BhaveshHeliconia BhaveshHeliconia BhaveshHeliconia approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved merged 🎉 ready to merge

Projects

None yet

Milestone

18.0

Development

Successfully merging this pull request may close these issues.