OCA / rma

Odoo for Return Merchandise Authorization (RMA)
GNU Affero General Public License v3.0
78 stars 206 forks source link

[16.0][MIG] rma_sale: Migration to 16.0 #364

Closed pedrobaeza closed 1 year ago

pedrobaeza commented 1 year ago

@Tecnativa TT44214

pedrobaeza commented 1 year ago

/ocabot migration rma_sale

pedrobaeza commented 1 year ago

Ouch, I looked for o2m in views, but not in wizards. Fixed.

About the portal JS, please do the needed changes, and it would be good to have a tour test for checking it.

chienandalu commented 1 year ago

I just tried this PR https://github.com/OCA/openupgradelib/pull/338 to migrate the qweb templates and it looks that its working nicely :) I added a commit to the PR

pedrobaeza commented 1 year ago

Is it correct then? Are you going to prepare a tour test?

chienandalu commented 1 year ago

Is it correct then? Are you going to prepare a tour test?

No, I should fix the delivery address selector in the portal. I can prepare a tour, yes.

pedrobaeza commented 1 year ago

Is this finished then?

chienandalu commented 1 year ago

Yes :)

pedrobaeza commented 1 year ago

I have improved a bit more the tests coverage and fine-tune several warnings, so it's time to merge it:

/ocabot merge nobump

OCA-git-bot commented 1 year ago

On my way to merge this fine PR! Prepared branch 16.0-ocabot-merge-pr-364-by-pedrobaeza-bump-nobump, awaiting test results.

OCA-git-bot commented 1 year ago

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