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

[10.0] mail_sendrid, Error: No module named simplejson #261

Closed
cuongnmtm opened this issue Apr 19, 2018 · 3 comments
Closed

[10.0] mail_sendrid, Error: No module named simplejson #261

cuongnmtm opened this issue Apr 19, 2018 · 3 comments
Assignees
@ecino
Labels
stale PR/Issue without recent activity, it'll be soon closed automatically.

Comments

@cuongnmtm
Copy link

cuongnmtm commented Apr 19, 2018 •
edited
Loading

File ".../addons/oca-social/mail_sendgrid/controllers/json_request.py", line 4, in
import simplejson
ImportError: No module named simplejson

I put simplejson into file requirements.txt and it solved this issue.
@pedrobaeza
Copy link
Member

Version and module in the title, please.

ecino added a commit to CompassionCH/social that referenced this issue Apr 19, 2018
@ecino
Rewrite dependencies of sendgrid module fixes OCA#261
1f2336f
ecino added a commit to CompassionCH/social that referenced this issue Apr 19, 2018
@ecino
Rewrite dependencies of sendgrid module fixes OCA#261
4de4c4b
@ecino
Copy link
Contributor

ecino commented Apr 19, 2018

done in #262

ecino added a commit to CompassionCH/social that referenced this issue Apr 20, 2018
@ecino
Rewrite dependencies of sendgrid module fixes OCA#261
fa5ef8a
@cuongnmtm cuongnmtm changed the title Error: No module named simplejson [10.0] mail_sendrid, Error: No module named simplejson Apr 20, 2018
pedrobaeza added a commit that referenced this issue May 18, 2018
@pedrobaeza
Merge pull request #262 from CompassionCH/10.0-sendgrid
ce2ea61 
Rewrite dependencies of sendgrid module fixes #261
Nooka10 pushed a commit to Nooka10/social that referenced this issue Sep 27, 2019
@ecino @Nooka10
Rewrite dependencies of sendgrid module fixes OCA#261
12861c0
Nooka10 pushed a commit to Nooka10/social that referenced this issue Nov 15, 2019
@ecino @Nooka10
Rewrite dependencies of sendgrid module fixes OCA#261
30a5f82
Nooka10 pushed a commit to Nooka10/social that referenced this issue Nov 15, 2019
@ecino @Nooka10
Rewrite dependencies of sendgrid module fixes OCA#261
58ba1f6
Nooka10 pushed a commit to Nooka10/social that referenced this issue Jan 6, 2020
@ecino @Nooka10
Rewrite dependencies of sendgrid module fixes OCA#261
91b0bc2
quentingigon pushed a commit to quentingigon/social that referenced this issue Mar 3, 2020
@ecino @quentingigon
Rewrite dependencies of sendgrid module fixes OCA#261
a77b724
quentingigon pushed a commit to quentingigon/social that referenced this issue Mar 3, 2020
@ecino @quentingigon
Add Sendgrid module from v9
3f92085 
[10.0] Migrate sendgrid modules

CO-1192 The mail are now correctly sent to all recipients

Sendgrid code corrections for v10 + Add tests

Rewrite dependencies of sendgrid module fixes OCA#261

[UPD] Update mail_sendgrid.pot

[10.0][FIX] mail_sendgrid_mass_mailing: Bad import (OCA#288)

[UPD] Update mail_sendgrid.pot

[FIX] mail_sendgrid*. Use info loglevel for import error.

Using _logger.warning for ImportError exception results in runbot/travis failures.

Add indexes to speed up performance

[IMP] use json library instead of simplejson

migrate mail_sendgrid to v11. Add support of sendgrid v3 API.

Add pre-migration script and update readme

No more commit in DB when tests are enabled

FIX: Ttracking event modify also the mail.mail status

FIX multiple body fields that were conflicting

- 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

FIX flake8 issue
@github-actions
Copy link

github-actions bot commented Dec 5, 2021

There hasn't been any activity on this issue in the past 6 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 issue 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 Dec 5, 2021
em230418 pushed a commit to em230418/social that referenced this issue Feb 8, 2022
@ecino @em230418
Rewrite dependencies of sendgrid module fixes OCA#261
cd5609c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ecino ecino

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

No branches or pull requests
3 participants
@pedrobaeza @ecino @cuongnmtm