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

initial commit for added email notifications for admins and staff mem… #519

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

nuukee
Copy link

@nuukee nuukee commented Oct 5, 2021

Hi, hier mal mein erster Versuch für die Geschichte mit admins und Mitarbeiter sollen auch emails bekommen

@tkrebs
Copy link
Owner

tkrebs commented Oct 6, 2021

Danke für dein Engagement und den Beitrag. 👍

Ich denke, das könnte für einige recht nützlich sein.

Inwieweit es für die breite Masse in die Main-Branch aufgenommen werden kann, muss ich mir noch anschauen.

@nuukee
Copy link
Author

nuukee commented Oct 7, 2021

Alles klar, danke. Ich habe versucht es so zu implementieren, dass das alte Verhalten nicht geändert wird. Man muss es ja explizit aktivieren.
Der Rest sind nur ein paar Namensänderungen und das Event-Triggering für die emails sowie die Erweiterung des email Codes für repeat bookings (was für das "cancel" event interessanterweise schon halb vorbereitet war, nur nie benutzt). Falls was unklar ist bitte melden :-)
Würde mich freuen wenn es in den Master Branch geht, macht mir das Leben natürlich einfacher.

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.

None yet

2 participants