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

[REF] copier Upate bump to v1.20 (from v1.17). #1134

Merged
merged 1 commit into from
Jan 18, 2024

Conversation

legalsylvain
Copy link
Contributor

[REF] Do not use OCB version, to save resources

@legalsylvain legalsylvain added this to the 16.0 milestone Jan 18, 2024
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think you should save resources this way, and more in a recent version. And in any case, you should opt for OCB, not Odoo.

@legalsylvain
Copy link
Contributor Author

legalsylvain commented Jan 18, 2024

I don't think you should save resources this way, and more in a recent version.

Hi @pedrobaeza, thanks for your quick answer.

  • I didn't know that OCA policy.
  • I see the OCB project as abandonned by OCA. in V16, there is 3 merged commits (differents than in odoo/odoo) : https://github.com/OCA/OCB/pulls?q=is%3Apr+base%3A16.0 : One to flag the repo as OCB and two regarding autoinstallation of iap.

And in any case, you should opt for OCB, not Odoo.

Well, we should ensure that OCA modules are working with odoo, not OCB. (even if it is preferable). odoo is used by millions of users, OCB, by some specific integrators.

@pedrobaeza
Copy link
Member

Well, it's not abandoned, it's simply that all the patches has landed on Odoo, but at the end OCB is more completed than Odoo, if you remove one of both. Take also into account that people has not yet landed too much on v16, so there's not a lot of PRs.

@legalsylvain
Copy link
Contributor Author

Hi @pedrobaeza. thanks for your review. I reverted the change regarding OCB/odoo, and I opened a dedicated issue in the right place : OCA/oca-addons-repo-template#244

@legalsylvain legalsylvain merged commit 589c529 into OCA:16.0 Jan 18, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants