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.

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

[MIG] mail_tracking_mailgun: Migrated to 10.0 #111

Merged
merged 1 commit into from
Nov 29, 2016
Merged

[MIG] mail_tracking_mailgun: Migrated to 10.0 #111

merged 1 commit into from
Nov 29, 2016

Conversation

bouvyd
Copy link
Contributor

@bouvyd bouvyd commented Oct 14, 2016

Note that this will have a red runbot until #108 is merged.

Not tested besides python test cases, I don't have a mailgun account nor a domain with an odoo server on it to test this fully. If your runbot instances stays alive long enough, we could probably test it from there once it'll build, not sure.

@pedrobaeza pedrobaeza mentioned this pull request Oct 14, 2016
21 tasks
@lasley lasley added this to the 10.0 milestone Oct 14, 2016
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

code review (this one is very independent from Odoo itself, as it depends on the mail_tracking webhooks)

Copy link
Contributor

@lasley lasley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

code LGTM as well. Gotta love when the migrations are this clean, will make forward/backports nice and easy 😄

@pedrobaeza
Copy link
Member

Merging this one also now that mail_tracking has been merged.

@pedrobaeza pedrobaeza merged commit 0003abf into OCA:10.0 Nov 29, 2016
@lasley
Copy link
Contributor

lasley commented Nov 29, 2016

Hooray just in time for my internal v10 migration!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants