CntoDev / rooster

Community Roster and related automation services for managing members
0 stars 0 forks source link

Warnings system #64

Closed enrico-ghidoni closed 2 years ago

enrico-ghidoni commented 2 years ago

There are several warnings to be shown to both Interviewers and Member Managers.

I'm sure there are going to be many other warnings that are either already in the django-roster or that could be useful to introduce. But I would stop with the three listed above for the MVP.

For the UX bit, different roles should see different warnings and only relative to their branch. For example, Interviewers should not be able to see warnings for Member Managers and viceversa. Also, displaying the warnings on the frontend should allow for some basic search and filtering features.

@milivojm if you agree this is enough for the MVP I can go ahead and split this into different sub-tasks.

milivojm commented 2 years ago

Warnings concept is obsolete. If we automate, we automate. If not, we can do this in Excel.

enrico-ghidoni commented 2 years ago

Not really. As I mentioned there are a bunch of different warnings for different things (i.e. contributor tags) and as explained in #62 I do believe that we should build automation at a later stage and keeping the system as self-explanatory as possible. So even if we end up automating things, a "notice board" should be there to inform staff about what has been automatically done and why

enrico-ghidoni commented 2 years ago

Moving discussion about automation to #68

milivojm commented 2 years ago

I've automated warning conditions described above, you can close it.

milivojm commented 2 years ago

Do we need this for MVP? Or at all?