leelokas / vacation_tracker

3 stars 3 forks source link

Improve notifications logic #111

Closed leelokas closed 7 years ago

leelokas commented 7 years ago

Test-users have said that:

lucyhilltop commented 7 years ago

When user edits a request, the notification should also say 'send to confirmation' like it does, when creating new request

leelokas commented 7 years ago

Sick leave type requests do not need manager confirmation so the notification shouldn't say it was sent for confirmation. In addition, the alert currently displays "01/01/1970" as end date when the end date is missing.

leelokas commented 7 years ago

Alerts section is cleared when user changes between 'My vacations', 'Confirmation needed', 'Overview', 'Settings' or 'User Management' pages.