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

feat: Sendgrid - activity feed #355

Merged
merged 7 commits into from
Apr 11, 2022
Merged

feat: Sendgrid - activity feed #355

merged 7 commits into from
Apr 11, 2022

Conversation

SanjuEpic
Copy link
Contributor

@SanjuEpic SanjuEpic commented Feb 23, 2022

Hey @FlorentLvr!
would like to ask you to review an issue that I'm facing currently via filtering emails for getting the stats from sendgrid (related to authorization error)
Here is the issue #53

Thanks!

@SanjuEpic SanjuEpic self-assigned this Feb 23, 2022
@SanjuEpic SanjuEpic changed the title Getting an authorization error feat: Sendgrid-Get_activity_feed Feb 23, 2022
@SanjuEpic SanjuEpic changed the title feat: Sendgrid-Get_activity_feed feat: Sendgrid- activity feed Feb 23, 2022
@SanjuEpic SanjuEpic changed the title feat: Sendgrid- activity feed feat: Sendgrid - activity feed Feb 23, 2022
@jravenel jravenel linked an issue Feb 25, 2022 that may be closed by this pull request
@Dr0p42
Copy link
Contributor

Dr0p42 commented Mar 8, 2022

Hey @FlorentLvr! would like to ask you to review an issue that I'm facing currently via filtering emails for getting the stats from sendgrid (related to authorization error) Here is the issue #53

Thanks!

Hello @SanjuEpic, I tried locally (@FlorentLvr tried as well) and I do have the same error as you. After looking at the documentation here it seems that:

You must purchase additional email activity history to gain access to the Email Activity Feed API.

So we just bought it ourselves and it seems that it will take a couple of hours to be active.

I will keep you posted.

Thank you for your contribution 💪

@SanjuEpic
Copy link
Contributor Author

Great @Dr0p42 and @FlorentLvr !!
Was thinking the same way,
Thank you for letting me know!

@SanjuEpic
Copy link
Contributor Author

SanjuEpic commented Mar 18, 2022

Hey @Dr0p42 @FlorentLvr!
Can you let me know how to access this email API (if there are any specific options to tweak)

Currently getting the error "BadRequestsError: HTTP Error 400: Bad Request"
while executing the Sendgrid Get activity feed notebook

error

Thanks!

@SanjuEpic
Copy link
Contributor Author

Any help regarding this step would be appreciated
@Dr0p42 do let me know about the API access credentials if any are required for the one in which the purchase was made.

Thanks!

@SanjuEpic
Copy link
Contributor Author

I have got to know some information regarding the SendGrid API
Have a look at this comment

Even when I tried with the URL in no encoded fashion, still I was getting the error but with a different message
"InternalServerError: HTTP Error 500: Internal Server Error" and that too it takes some time (about 1 min of execution for this cell)

Possible debug options:

  1. Get the right from email to filter i.e the mail id that is used for authorizing the sendgrid app at the time of purchase of the email activity feed API and then debug

Any suggestions are welcome
Thanks!
server_error

@jravenel
Copy link
Contributor

The comment is in the Ruby version of the API no ? Same for python ?

@SanjuEpic
Copy link
Contributor Author

Yeah I tried it but no results I have got with respect to python. :(

@FlorentLvr
Copy link
Contributor

Yeah I tried it but no results I have got with respect to python. :(

I used requests library to get all messages and it's working now. Could you please confirm on your side too ? :)

@SanjuEpic
Copy link
Contributor Author

Okay, that's a great help for this issue, will have a look @FlorentLvr.
Thanks!

@Dr0p42
Copy link
Contributor

Dr0p42 commented Apr 7, 2022

Sorry @SanjuEpic I got sidetracked a loooot. Let me know if you need help on this one ;)

@SanjuEpic
Copy link
Contributor Author

Sure will let you know @Dr0p42
Thanks

@SanjuEpic
Copy link
Contributor Author

I have gone through the code @FlorentLvr, it's running on my side as well.
We can merge this PR now!
Thanks for the help!

Copy link
Contributor

@Dr0p42 Dr0p42 left a comment

Choose a reason for hiding this comment

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

Well done @SanjuEpic 👏

Thank you for this contribution :)

@Dr0p42 Dr0p42 merged commit db39a3d into master Apr 11, 2022
@Dr0p42 Dr0p42 deleted the sendgrid_activity_feed branch April 11, 2022 14:50
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.

Sendgrid - Get activity feed as dataframe
5 participants