✨ Added PollResponse Through Email #108
Conversation
|
Congrats and thanks! |
|
@PushkarDureja I'll take a look once I reach home (in couple hours). |
|
We will merge it once the endpoint is done. |
|
Great !! |
|
@PushkarDureja have you tested this PR with the new endpoint locally? If yes, we can merge this. |
Yes I did it yesterday only after adding back the |
|
Thanks, merging! |
|
We've deployed this PR to our site, so can you check out the changes and let us know if everything is fine? |
Sure, Let me check |
|
@anandbaburajan I don't think it is working!! Did you redeploy the mailer ? |
Yup. |
|
As far as I remember, It was working fine on my local environment. But give me some time to figure it out |
|
@AthulJoshi can you share the mailer logs with Pushkar and/or look into it? Thanks! |
| senderName: newVote.name, | ||
| }; | ||
| sendPollResponse(mailerArgs, token); | ||
| } | ||
| Router.reload(); |
There was a problem hiding this comment.
After commenting line 57 I took a screenshot of network logs and It shows the pollresponse request is working fine
|
@AthulJoshi I have added a comment, Please have a look |
|
So, the page is reloading before the request is completed right? |
But even if the request has been initiated (which is actually happening, I checked on the live one), Will it depend on the fact, whether the browser is being reloaded or not ? |
Fixes #95
I have implemented the code for sending email to the poll creator whenever someone votes on their poll