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

[11.0] migrate mail_sendgrid module #486

Closed
wants to merge 17 commits into from
Closed

[11.0] migrate mail_sendgrid module #486

wants to merge 17 commits into from

Conversation

ecino
Copy link
Contributor

@ecino ecino commented Jan 15, 2020

  • uses new python sendgrid API

ecino and others added 15 commits January 6, 2020 13:44
@ecino @Nooka10
Add Sendgrid module from v9
81ccfd7
@ecino @Nooka10
[10.0] Migrate sendgrid modules
de8a7ad 
CO-1192 The mail are now correctly sent to all recipients
@ecino @Nooka10
Sendgrid code corrections for v10 + Add tests
cf098ea
@ecino @Nooka10
Rewrite dependencies of sendgrid module fixes OCA#261
91b0bc2
@oca-travis @Nooka10
[UPD] Update mail_sendgrid.pot
9de5de8
@igallart @Nooka10
[10.0][FIX] mail_sendgrid_mass_mailing: Bad import (OCA#288)
6d85d8f
@oca-travis @Nooka10
[UPD] Update mail_sendgrid.pot
af2fe48
@NL66278 @Nooka10
[FIX] mail_sendgrid*. Use info loglevel for import error.
aa00d42 
Using _logger.warning for ImportError exception results in runbot/travis failures.
@ecino @Nooka10
Add indexes to speed up performance
9507a08
@hbrunn @Nooka10
[IMP] use json library instead of simplejson
04faa7e
@Nooka10
migrate mail_sendgrid to v11. Add support of sendgrid v3 API.
bbc18d6
@Nooka10
Add pre-migration script and update readme
3e60ce4
@Nooka10
No more commit in DB when tests are enabled
86bdf84
@Nooka10
FIX: Ttracking event modify also the mail.mail status
92a8a9c
@ecino
FIX multiple body fields that were conflicting
9d61e9e 
- Instead of copying body of template to mass mailing, directly use
  the template text for mass mailings when using Sendgrid.
- If needed, the template must be edited instead of the mass mailing.
- This simplifies the code complexity
@ecino ecino mentioned this pull request Jan 15, 2020
Closed
26 tasks
@Nooka10 Nooka10 mentioned this pull request Jan 15, 2020
Closed
@github-actions
Copy link

github-actions bot commented Apr 3, 2022

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Apr 3, 2022
@github-actions github-actions bot closed this May 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@simahawk simahawk Awaiting requested review from simahawk

@rafaelbn rafaelbn Awaiting requested review from rafaelbn

Assignees
No one assigned
Labels
stale PR/Issue without recent activity, it'll be soon closed automatically.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@ecino @oca-travis @igallart @NL66278 @hbrunn @Nooka10