Contributors mailing list archives
contributors@odoo-community.org
Browse archives
Re: new API vs merges upwards
by
Pledra Netcom, Paul Catinean
I'm with Pedro on this one, +1
Regards.
On Fri, Jul 31, 2015 at 1:08 PM, <Pedro@pad.odoo-community.org> wrote:
In some moment in the future, we need to break with old API, and forget to make merge upwards from 7 to 8. We can delay this moment, but when v9 wil be out, we shouldn't maintain the chain 7 > 8 > 9. I vote for breaking already in v8, and make 8 > 9 merge upwards, because new API brings a lot of new possibilities.Regards.2015-07-31 11:52 GMT+02:00 Leonardo Pistone <lpistone@gmail.com>:We hit this topic other times, but this time we have a practical example: https://github.com/OCA/stock-logistics-workflow/pull/132/files#diff-cfbfcae8295adcadf086ea7bfa09b21dL185 What happened is that the v7 of a module was improved, and in the merge upwards of 7 to 8, I resolved the conflict choosing the v7 version. In that specific case I thought that I was just throwing away two trivial new API lines for a big win. I do not think there is a good solution to that. Even the solution of porting the method to v8 is not ideal: the fresh v7 code is in production somewhere, and if it breaks we'll sure post a fix, bringing us back at square one. The new API port will be unused by us for the moment, hiding new bugs for a while. Odoo SA does upwards merges regularly, but only because they did very little porting to the new API. In our case, I'm not sure if it's worth it. What do you think on the subject?_______________________________________________
Mailing-List: http://odoo-community.org/groups/oca-contributors-15
Post to: mailto:contributors@odoo-community.org
Unsubscribe: http://odoo-community.org/groups?unsubscribe_______________________________________________
Mailing-List: http://odoo-community.org/groups/oca-contributors-15
Post to: mailto:contributors@odoo-community.org
Unsubscribe: http://odoo-community.org/groups?unsubscribe
Reference
-
new API vs merges upwards
byLeonardo Pistone-
Re: new API vs merges upwards
byOpen Architects Consulting, Houssine BAKKALI