Open-event-server: After-Event Email: Several identical mails sent out, if user has several sessions/roles

[mariobehling]

If a user has several sessions and roles, e.g. speaker and event organizer, several after-event emails are sent out. Expected: Only send one email per email address.

[kushthedude]

@iamareebjamal server issue

[iamareebjamal]

Bug in APS, to be replaced by celery beat. Every gunicorn worker including celery runs a copy of APS, and hence everything is repeated 5 times (4 workers + 1 celery)

[kushthedude]

APS: Python Scheduler ? I will be looking into it.

[codedsun]

Taking this up! 👍

[kushthedude]

APS: Python Scheduler ? I will be looking into it.

Also there are already 3-4 PR opened by you which are not merged. Please work on them before making more new one's.

[codedsun]

Also there are already 3-4 PR opened by you which are not merged. Please work on them before making more new one's.

It has been 3 days and no one was working and the priority is HIGH also.
@kushthedude If you are working on this, then no problem. Also yes they are in the pending state due to the solution being searched simultaneously as I m stuck in both and in sync with Areeb (You can see the conversation). And if you are not working on this, then I can send the patch, why not move the project forward simultaneously. Please tell what should be done?

Thanks.

[kushthedude]

There are a lot of issues already present,needless to say you should give
atleast a week to other dev before claiming the issue if they are working
on it. If you dont hear from them a week or dont see a patch you should
definitely go ahead after commenting.
Each contributor should get an equal opportunity for contribution whether
it be me, you or any new comer.
Anyway, If you don't have issue to work on, you can go ahead with it. No
issues 😄

On Fri, 10 Jan, 2020, 22:20 Suneet Srivastava, [e-mail]
wrote:

Also there are already 3-4 PR opened by you which are not merged. Please
work on them before making more new one's.

It has been 3 days and no one was working and the priority is HIGH also.
@kushthedude https://github.com/kushthedude If you are working on this,
then no problem. Also yes they are in the pending state due to the solution
being searched simultaneously as I m stuck in both and in sync with Areeb
(You can see the conversation). And if you are not working on this, then I
can send the patch, why not move the project forward simultaneously. Please
tell what should be done?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fossasia/open-event-server/issues/6731?email_source=notifications&email_token=AKQMTLSD5G3ZLZAAWPSSZT3Q5CRMPA5CNFSM4KEER33KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIUQSKQ#issuecomment-573114666,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKQMTLTXERWZOAYMVUWWLXDQ5CRMPANCNFSM4KEER33A
.

[vishakhanihore]

@codedsun Are you working on it?

[codedsun]

No, @vishakhanihore You may ask @kushthedude if he is working on it.

[kushthedude]

Sure go ahead.

On Sat, 11 Jan, 2020, 09:48 Suneet Srivastava, [e-mail]
wrote:

No, @vishakhanihore https://github.com/vishakhanihore You may ask
@kushthedude https://github.com/kushthedude if he is working on it.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fossasia/open-event-server/issues/6731?email_source=notifications&email_token=AKQMTLXVPYDKWTVLRBZFE4DQ5FB77A5CNFSM4KEER33KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIVY2RY#issuecomment-573279559,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKQMTLVZKMQUEXAME3RPH6TQ5FB77ANCNFSM4KEER33A
.

[codedsun]

@vishakhanihore - Are you still working on this?

[vishakhanihore]

Didnt get time to finish it. You may go ahead if you want.

On Sun, 19 Jan, 2020, 18:35 Suneet Srivastava, [e-mail]
wrote:

@vishakhanihore https://github.com/vishakhanihore - Are you still
working on this?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fossasia/open-event-server/issues/6731?email_source=notifications&email_token=AM6PS4WDCABD4SW6FFVBRP3Q6RFZBA5CNFSM4KEER33KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJKRUZQ#issuecomment-576002662,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AM6PS4WMNTIMMBWNXQPQI5DQ6RFZBANCNFSM4KEER33A
.

[codedsun]

@iamareebjamal - In this, all the scheduled jobs will become a celery task and then be scheduled by celery beat. correct?

[iamareebjamal]

yes

[kushthedude]

When sending mail function is called, it sends to all speaker, organiser, owner irrespective of if it has already been sent to one.

[codedsun]

@kushthedude - Yes, but it runs after every 5:30 mins. Can you point more clearly what you want to say.

Also, @iamareebjamal mentioned that it was due to the gununicorn workers(4 in numbers) which were causing this issue, that's why we replaced the APS with celery beat.

[kushthedude]

@kushthedude - Yes, but it runs after every 5:30 mins. Can you point more clearly what you want to say.

I didn't know why you even mentioned this.

See this snippet

 if current_time > event.ends_at and time_difference_minutes < 1440:
                for speaker in speakers:
                    if not speaker.is_email_overridden:
                        send_email_after_event(speaker.user.email, event.name, frontend_url)
                        send_notif_after_event(speaker.user, event.name)
                for organizer in organizers:
                    send_email_after_event(organizer.user.email, event.name, frontend_url)
                    send_notif_after_event(organizer.user, event.name)
                if owner:
                    send_email_after_event(owner.user.email, event.name, frontend_url)
                    send_notif_after_event(owner.user, event.name)

[kushthedude]

@kushthedude - Yes, but it runs after every 5:30 mins. Can you point more clearly what you want to say.

I didn't know why you even mentioned this.

See this snippet

 if current_time > event.ends_at and time_difference_minutes < 1440:
                for speaker in speakers:
                    if not speaker.is_email_overridden:
                        send_email_after_event(speaker.user.email, event.name, frontend_url)
                        send_notif_after_event(speaker.user, event.name)
                for organizer in organizers:
                    send_email_after_event(organizer.user.email, event.name, frontend_url)
                    send_notif_after_event(organizer.user, event.name)
                if owner:
                    send_email_after_event(owner.user.email, event.name, frontend_url)
                    send_notif_after_event(owner.user, event.name)

@iamareebjamal What should be the optimal approach for this, A bunch of conditionals with flag variable or something else?

[iamareebjamal]

Extract all emails in a set and loop over them to send the email

[kushthedude]

There is more this issue, being ab owner and Speaker of a event. I just
received 15 after event mail

On Fri, 31 Jan, 2020, 06:14 Areeb Jamal, [e-mail] wrote:

Extract all emails in a set and loop over them to send the email

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fossasia/open-event-server/issues/6731?email_source=notifications&email_token=AKQMTLTOKWTU24CV2MRMBZ3RANX6RA5CNFSM4KEER33KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKNC2UA#issuecomment-580529488,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKQMTLR2IVK2RXJ6MCOC2STRANX6RANCNFSM4KEER33A
.