search

Vauxoo / addons-vauxoo

All our modules related to developments that solves generic issues on Odoo, or that solve internal problems on Odoo Core, if something is here, maybe it is solving an issue in your company, try it and report what you see.
http://www.vauxoo.com
193 stars 288 forks source link

[MIG] web_widget_json_graph: migration to 12.0 #1465

Closed luisg123v closed 3 years ago

coveralls commented 3 years ago

Coverage Status

Coverage remained the same at 58.589% when pulling d284c029cba40b66e8cf0bfe33dd7c1ea629ad0b on vauxoo-dev:12.0-mig_web_widget_json_graph-luisg into 8c9a1597b4e3bd3904ceca75ecd8cef24c7862ec on Vauxoo:12.0.

luisg123v commented 3 years ago

@moylop260 @hugho-ad @frahikLV

could you review, please?

I'm migrating this module from the web repo, because that module was actually never in the OCA, so ours is diverging without needing to.

hugho-ad commented 3 years ago

why not post it at oca/web?

luisg123v commented 3 years ago

@hugho-ad because it has never belonged there.

It could be added, but it would be a new module.

hugho-ad commented 3 years ago

@luisg123v

It could be added, but it would be a new module. New module name?

I think this we should move it there, at oca/web

luisg123v commented 3 years ago

@hugho-ad same module name, but new module for the OCA

moylop260 commented 3 years ago

I think It is not a bad idea to move this module to OCA

luisg123v commented 3 years ago

@moylop260 @hugho-ad PR to the OCA created: https://github.com/OCA/web/pull/1733. Authorship has been taken care of.

I'm closing this one.