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

Secretaries are getting notification emails for providers that are not assigned to them. #966

Closed
mbaldazzi85 opened this issue Jan 5, 2021 · 3 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@mbaldazzi85
Copy link

When an appointment is made for a provider with secretary and the email notifications are active only for the secretary (not for the provider), an appointment notification email is received also for another provider that has notifications active but which is not connected with the requested provider.
From / admin we have verified that the notification flags are set regularly.
the version used is 1.4.1 stable.

@alextselegidis alextselegidis self-assigned this Jan 8, 2021
@alextselegidis
Copy link
Owner

Hello!

Thanks for reporting, this is indeed a bug.

There's a typo in the notifications library that leads to sending mails to secretaries for providers that are not connected to them, while they do not receive for the providers that are connected.

Will be fixed with the next release, till them you can use the fix of the attached commit.

  Alex Tselegidis, Easy!Appointments Creator
  Need a customization? Contact me in person!

@alextselegidis alextselegidis added the bug Something isn't working label Jan 8, 2021
@alextselegidis alextselegidis changed the title notification email forward appointments to wrong provider Secretaries are getting notification emails for providers that are not assigned to them. Jan 8, 2021
@alextselegidis alextselegidis added this to the v1.4.2 milestone Jan 8, 2021
@RaphaelWimmer
Copy link

This also fixes #262 now.

@PrDigitus
Copy link

PrDigitus commented Jul 21, 2021

Hi,
I am on 1.4.1 now, but still have the Issue, that Secretary receives no mails.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants