Skip to content

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

Open
nuukee wants to merge 5 commits into
tkrebs:masterfrom
nuukee:custom-notifications-for-staff
Open

initial commit for added email notifications for admins and staff mem…#519
nuukee wants to merge 5 commits into
tkrebs:masterfrom
nuukee:custom-notifications-for-staff

Conversation

@nuukee

@nuukee nuukee commented Oct 5, 2021

Copy link
Copy Markdown

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

@tkrebs

tkrebs commented Oct 6, 2021

Copy link
Copy Markdown
Owner

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

nuukee commented Oct 7, 2021

Copy link
Copy Markdown
Author

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.

2 participants