-
-
Notifications
You must be signed in to change notification settings - Fork 479
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
[12.0][MIG] website_sale_checkout_country_vat #286
[12.0][MIG] website_sale_checkout_country_vat #286
Conversation
…e in website checkout form.
Replace settings with sane defaults. Update VAT flag when address country is updated.
If you leave country field empty before pressing the confirm button at checkout, you were getting a 500 error because `country_id` was `""` (str), which cannot be searched in an int field. Now we set a default `False` value, which will return an empty recordset without that problem.
0aa8544
to
eb2681d
Compare
Runbot fails because missing dependencies.... (are in other repo and not merged into 12.0) |
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.
Trying on runbot, I find these problems:
- When not logged in, pressing "Sign up and checkout" leads to
404
page - When logged in by other means, if I edit the address, I get
500: Internal Server Error
.
About Travis tests, can be due to this module. |
24991f4
to
dcc7c09
Compare
I still get 404 page when I click in "Sign up and checkout". Is it due to this module? I have been able however to see the country dropdown when editing the address once logged-in. |
dcc7c09
to
203e4be
Compare
Dropped oca_dependencies changes. |
203e4be
to
7e11ef6
Compare
Reverted "$target" changes |
@pedrobaeza The problem is that "website_sale_suggest_create_account" module fails on runbot... I installed it on local and works :/ |
Please check with @yajo what can be the problem. |
Although I'm now seeing that this PR is green. |
Normal migration... Odoo 12.0 doesn't have "vat" as required field.