Skip to content
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

[17.0][OU-ADD] payment: Migration scripts #4584

Open
wants to merge 2 commits into
base: 17.0
Choose a base branch
from

Conversation

remi-filament
Copy link
Contributor

@remi-filament remi-filament commented Oct 11, 2024

Take over payment migration from PR #4466 upon @duong77476-viindoo request (#4466 (comment))

Extra tasks performed :

  • Remove payment.payment_method_acss_debit from noupdate_changes since this module does not exist anymore
  • Rename model payment.icon to payment.method
  • Reload payment methods

@rvalyi
Copy link
Member

rvalyi commented Oct 11, 2024

for reference, the 5 key commits for v17: https://github.com/akretion/odoo-module-diff-analysis/tree/main/17.0/payment

@remi-filament remi-filament force-pushed the 17.0-payment branch 2 times, most recently from d7cf356 to 0944bd0 Compare October 18, 2024 09:10
haumenphai and others added 2 commits October 18, 2024 11:11
Remove payment.payment_method_acss_debit as does not exist anymore
Move payment.icon to payment.method
Reload payment methods and links to providers
@remi-filament
Copy link
Contributor Author

Thanks @rvalyi it helps understanding where the changes come from (although in that case many changes were on data so there are more commits to take into account).
I added migration script to transform payment.icon in payment.method, even if I am not sure it was worth the effort since they changed pretty much everything, it would probably have been better to just drop everything and let ORM recreate proper values...

@legalsylvain
Copy link
Contributor

legalsylvain commented Oct 24, 2024

/ocabot migration payment

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Oct 24, 2024
@OCA-git-bot

This comment was marked as outdated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants