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][MIG] sale_discount_display_amount: Migration to 17.0 #2987

Closed

Conversation

FSchiepers
Copy link

This version adds the possibility to choose whether the discount is calculated based on the non-taxed subtotal or the taxed total, see: #2813

@FSchiepers FSchiepers changed the title [MIG] sale_discount_display_amount: Migration to 17.0 [17.0][MIG] sale_discount_display_amount: Migration to 17.0 Mar 1, 2024
@FSchiepers FSchiepers mentioned this pull request Mar 1, 2024
68 tasks
@FSchiepers
Copy link
Author

/ocabot migration sale_discount_display_amount

@OCA-git-bot
Copy link
Contributor

Sorry @FSchiepers you are not allowed to mark the addon tobe migrated.

To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons.

If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the maintainers key of its manifest.

@rousseldenis
Copy link
Sponsor Contributor

/ocabot migration sale_discount_display_amount

@rousseldenis
Copy link
Sponsor Contributor

@FSchiepers Thanks for this. Could you follow this migration guide ? https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-17.0#technical-method-to-migrate-a-module-from-160-to-170-branch

You should respect commit history.

@FSchiepers
Copy link
Author

The guide is not complete, when I try to make a pull request I get a github error.

@FSchiepers FSchiepers closed this by deleting the head repository Mar 18, 2024
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.

None yet

3 participants