Skip to content

[9.0][MIG][base_custom_info] Migration. - #435

Merged
pedrobaeza merged 18 commits into
OCA:9.0from
Tecnativa:9.0-base_custom_info-migration
Jul 22, 2016
Merged

[9.0][MIG][base_custom_info] Migration.#435
pedrobaeza merged 18 commits into
OCA:9.0from
Tecnativa:9.0-base_custom_info-migration

Conversation

@yajo

@yajo yajo commented May 26, 2016

Copy link
Copy Markdown
Member

Mostly no changes from 8.0 after all.

@Tecnativa #271

@pedrobaeza pedrobaeza mentioned this pull request May 26, 2016
59 tasks
@coveralls

coveralls commented May 26, 2016

Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.2%) to 73.227% when pulling 313d0f7 on Tecnativa:9.0-base_custom_info-migration into c4f7a86 on OCA:9.0.

Mostly no changes from 8.0 after all.
@yajo
yajo force-pushed the 9.0-base_custom_info-migration branch from 313d0f7 to 4783c9f Compare May 26, 2016 14:58
@coveralls

coveralls commented May 26, 2016

Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.2%) to 73.227% when pulling 4783c9f on Tecnativa:9.0-base_custom_info-migration into c4f7a86 on OCA:9.0.

@moylop260

Copy link
Copy Markdown
Contributor

👍
thanks

@rafaelbn

Copy link
Copy Markdown
Member

👍

1 similar comment
@pedrobaeza

Copy link
Copy Markdown
Member

👍

@pedrobaeza
pedrobaeza merged commit 1572fa6 into OCA:9.0 Jul 22, 2016
@pedrobaeza
pedrobaeza deleted the 9.0-base_custom_info-migration branch July 22, 2016 07:16
@diggy128

diggy128 commented Nov 9, 2018

Copy link
Copy Markdown

Last commit from @yajo mentioned almost no changes to migrate to V9.0, however the merge from @pedrobaeza severely cripples the module removing several fields and changing the behavior with no visible migration path.

Was this an mentioned/inteded change or something got wrong?

@pedrobaeza

Copy link
Copy Markdown
Member

See the other PR for the explanation of the changes.

SiesslPhillip pushed a commit to grueneerde/OCA-server-tools that referenced this pull request Nov 20, 2024
Syncing from upstream OCA/server-tools (8.0)
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