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.

Sign up for GitHub

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

Jump to bottom

[ADD] crm: Migration scripts #828

Merged
merged 4 commits into from May 31, 2017
Merged

[ADD] crm: Migration scripts #828

merged 4 commits into from May 31, 2017

Conversation

pedrobaeza
Copy link
Member

  • Updated documentation
  • Analysis work file
  • Pre and post migration scripts
  • Tests

@pedrobaeza pedrobaeza changed the title 10.0 crm [ADD] crm: Migration scripts May 16, 2017
@pedrobaeza pedrobaeza mentioned this pull request May 16, 2017
Closed
@pedrobaeza pedrobaeza mentioned this pull request May 22, 2017
Merged
@pedrobaeza pedrobaeza force-pushed the 10.0-crm branch 7 times, most recently from 52e9d40 to 29c4dc3 Compare May 24, 2017 08:05
@pedrobaeza
Copy link
Member Author

@hbrunn can you help me to determine the root of the problem?

@hbrunn
Copy link
Member

hbrunn commented May 26, 2017

@pedrobaeza I'd hope so, but I feel quite clueless myself currently. Any chance the reference doesn't return what you expect it to return? (a missing id or the like?) Next idea would be to check if the registry gets botched somewhere on the way, did you inspect self.env['crm.lead']._fields['team_id'] if members like _setup_done contain what they should? Chances are we simply run our tests too early during initialization.

@StefanRijnhart
Copy link
Member

@pedrobaeza I took the liberty to commit what should be the fix of your test 0941fbc

@pedrobaeza
Copy link
Member Author

Travis is still red but it's due to sale_stock dependency without merging stock one. I merge and fix finally on stock migration.

@StefanRijnhart, sorry for letting this happens. I'm fixing it ASAP.

@pedrobaeza pedrobaeza merged commit b575b08 into OCA:10.0 May 31, 2017
@pedrobaeza pedrobaeza deleted the 10.0-crm branch May 31, 2017 07:48
@StefanRijnhart
Copy link
Member

@pedrobaeza Thank you for understanding, and thank you and the other developers for the great work that you are currently doing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@pedrobaeza @hbrunn @StefanRijnhart