Skip to content

Conversation

yajo
Copy link
Member

@yajo yajo commented May 26, 2016

Mostly no changes from 8.0 after all.

@Tecnativa #271

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

coveralls commented May 26, 2016

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
Copy link

coveralls commented May 26, 2016

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
Contributor

👍
thanks

@rafaelbn
Copy link
Member

👍

1 similar comment
@pedrobaeza
Copy link
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
Copy link

diggy128 commented Nov 9, 2018

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
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