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

Add an option for shifts to be "required" : if they are not filled X days before, specific accounts get notified. #491

Open
Theophile-Madet opened this issue Mar 8, 2024 · 5 comments
Assignees
Labels
big big task

Comments

@Theophile-Madet
Copy link
Contributor

For each shift, we should be able to specify how many days before the warnings goes out (maybe in hours? or "at 18:00 the day before"?) and to which accounts the mail gets sent to. The email address linked to those accounts should be used.

@Theophile-Madet Theophile-Madet added the big big task label Mar 8, 2024
@crosspolar crosspolar self-assigned this Apr 21, 2024
@crosspolar
Copy link
Contributor

grafik

@crosspolar
Copy link
Contributor

crosspolar commented Jul 27, 2024

Tests TODO

  • Every User can register to a ShiftWatch of a shift only once
  • Shift is deleted from ShiftWatch too when shift has been cancelled

@Theophile-Madet
Copy link
Contributor Author

@crosspolar do you have any work in progress on this? The PR is closed but I'm not sure if there's anything to keep from it?

@crosspolar
Copy link
Contributor

I found the requirement engineering a bit overwhelming, as you can see in the PR. I decided, that being able to choose which kind of mails to receive is obligatory for this, so I started programming this. The WIP is the PR itself basically, but I wasn't happy with the design.

@crosspolar
Copy link
Contributor

If you need a very basic method to notify just one mail adress with a fixed time ahead, it can be done much easier. But the PR aimed at members being able to "watch" shifts and get notified if the shift is understaffed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
big big task
Projects
Status: Backlog
Development

Successfully merging a pull request may close this issue.

2 participants