Skip to content

[MIG] github_connector_odoo: Migration to 12.0 - #38

Merged
pedrobaeza merged 22 commits into
OCA:12.0from
Tecnativa:12.0-mig-github_connector_odoo
Apr 11, 2019
Merged

pedrobaeza merged 22 commits into
OCA:12.0from
Tecnativa:12.0-mig-github_connector_odoo

Conversation

@cubells

@cubells cubells commented Apr 5, 2019

Copy link
Copy Markdown
Member

Create README using oca fragments

Standard migration

cc @Tecnativa

legalsylvain and others added 21 commits April 5, 2019 10:47
[FIX] module icon resize problem
* [FIX]  Fix field property

The property is 'readony'.

* [IMP]Get module icon from multiple sources

Get the module icon from the following sources (in order):
 1. From the first entry of the images tag from the manifest
 2. From the static directory
* [ADD] github_connector_odoo: New field category_id

[IMP] increase version number
[FIX] Lints

* [REF] removed unecessary variables.
[FIX] Lints

* [FIX] corrected version, since this update requires a module upgrade
* [IMP] Add try-except on branch analyze action

Add try-except statements on analyze actions that could
potentially crash the analyze process.

* [REF] Fix error logging

Changes include:
 - fix error values
 - use error instead of warning

* [IMP] Fix error in analyze call

Fix a programming mistake in the analyze call.
By @RoelAdriaans

* [IMP] Bump version number of github_connector

Bump the version number of github_connector to '11.0.1.1.1'.

* [IMP] Bump version number of github_connector

Bump the version number of github_connector_odoo to
'11.0.1.1.1'.
Set a default icon per module, when no icon is defined in that
module. The default icon is the 'OCA cube', the icon is located
in the module itself (in /data/oca.png).
…CA#30)

* [FIX]do not delete module version if already exists and just update

* [FIX]remove not needed code

* [IMP] added clean odoo module version cron

* [IMP]support module_paths also

* [IMP]separate function fo the module path to remove redundant code
To play nice with Sentry.
@cubells cubells changed the title 12.0 mig GitHub connector odoo [MIG] github_connector_odoo: Migration to 12.0 Apr 5, 2019
@pedrobaeza pedrobaeza added this to the 12.0 milestone Apr 5, 2019
@OCA-git-bot OCA-git-bot mentioned this pull request Apr 5, 2019
3 tasks
Comment thread github_connector_odoo/readme/CONFIGURE.rst Outdated
Comment thread github_connector_odoo/readme/ROADMAP.rst Outdated
Comment thread github_connector_odoo/views/view_odoo_license.xml Outdated
Comment thread github_connector_odoo/views/view_odoo_module.xml Outdated
Comment thread github_connector_odoo/views/view_odoo_module.xml Outdated
Comment thread github_connector_odoo/views/view_odoo_module_version.xml Outdated
Comment thread github_connector_odoo/views/view_odoo_module_version.xml Outdated
@pedrobaeza

Copy link
Copy Markdown
Member

You can squash a bit initial commit history by author.

@pedrobaeza

Copy link
Copy Markdown
Member

@victormmtorres can you take this PR and fix my comments as @cubells is now busy with other things?

@martintorresvm

Copy link
Copy Markdown

@pedrobaeza please have a check

Comment thread github_connector_odoo/readme/CONFIGURE.rst Outdated
Comment thread github_connector_odoo/views/view_odoo_license.xml Outdated
@martintorresvm
martintorresvm force-pushed the 12.0-mig-github_connector_odoo branch from 3652d62 to 9c37563 Compare April 11, 2019 09:53
@martintorresvm

Copy link
Copy Markdown

Changes done @pedrobaeza

@martintorresvm
martintorresvm force-pushed the 12.0-mig-github_connector_odoo branch from 9c37563 to 11d7653 Compare April 11, 2019 09:55
@martintorresvm
martintorresvm force-pushed the 12.0-mig-github_connector_odoo branch from 11d7653 to f02a38a Compare April 11, 2019 09:58
@pedrobaeza

Copy link
Copy Markdown
Member

Merging for continuing with the rest of the dependency chain.

@pedrobaeza
pedrobaeza merged commit af269e6 into OCA:12.0 Apr 11, 2019
@pedrobaeza
pedrobaeza deleted the 12.0-mig-github_connector_odoo branch April 11, 2019 19:11
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.

8 participants