-
-
Notifications
You must be signed in to change notification settings - Fork 390
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
Update crm_lead_address_street3 to fix Travis. #61
Conversation
I hope you can merge this ASAP to continue with all other PR. Anyway all are rebased on this now. |
@@ -21,31 +21,21 @@ | |||
from openerp import models, fields, api | |||
|
|||
|
|||
class crm_lead(models.Model): | |||
class Lead(models.Model): |
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.
Call if CrmLead
@StefanRijnhart, there was any reason to put @api.v7? |
We can ask the author @rdeheele but I don't think so. 👍 for now. What problem does @api.v7 cause in Travis? I can't check Travis in the linked PRs, mabye because they have been rebased. |
See all linked PR. |
Mostly #52 (comment). |
Thanks for the link, and for fixing this! 👍 |
@pedrobaeza @StefanRijnhart i don't remember exactly, there are cr and uid parameters, maybe it was easier to use api.v7 to inherit a core class itself in a v7 api style, 8 months ago. |
Really api.v7 had not much sense, since it is used only in conjuntion with api.v8 to provide different implementations for the same method depending on the calling method. Since there was no api.v8, all calls from v8 were failing. |
ok 👍 |
Update crm_lead_address_street3 to fix Travis.
Fix Travis for #52, #53, #54 and #55.
cc @pedrobaeza @rafaelbn