-
-
Notifications
You must be signed in to change notification settings - Fork 489
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
[11.0][MIG] website_sale_require_login: basic mig from v10 to v11 #224
[11.0][MIG] website_sale_require_login: basic mig from v10 to v11 #224
Conversation
Please one module per PR and take into account #222 |
will do two PRs, i can see #222 is approved, merge in order to rebase please |
355dc75
to
9b17cbd
Compare
Depends on #225 to satisfy dependency tree |
Usually OCA requires to keep original commit history. Check https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-11.0#technical-method-to-migrate-a-module-from-100-to-110-branch |
I was about to say it 😉 |
160f6c3
to
8d5634e
Compare
Example of module which requires such refactoring: https://github.com/it-projects-llc/website-addons/tree/10.0/website_sale_checkout_store [FIX] condition to show normal checkout button was wrong in website_sale_suggest_create_account I was equal to (user_authenticated or not signup_allowed and can_checkout) while it has to be (user_authenticated or not signup_allowed) and can_checkout
e602a6c
to
059bb23
Compare
059bb23
to
a456d9c
Compare
@yajo this is pretty much ready to review |
Functional and code review 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
code review and test
Depends on