bcgov / performance

Performance
Apache License 2.0
0 stars 2 forks source link

Add Dept ID list to overnight auto excuse process #1138

Closed Travis-A-Clark closed 12 months ago

Travis-A-Clark commented 1 year ago

Currently the overnight excuse process checks for variables including classification and PeopleSoft status to auto-excuse individuals from the PDP. This removes the individuals from PDP statistics and reports, pauses their next conversation due dates, and stops them from receiving email notifications.

We have heard from some business units that do not want to participate in the PDP at all. Let's add an additional check for Dept ID to the overnight excuse process to remove these individuals.

Also add a new option to the Excuse Reason list that says "Dept ID". So someone removed by this process would be Excused=Auto, Reason=Dept ID, Excused By=System.

Here is the list of Dept IDs to excuse. This list may change in the future.

Judiciary - Provincial Court 105-1115

Judiciary - Superior Court 105-1200

Initially also requested, but request was rescinded before ticket was deployed: 105-0751 105-1120 105-1125

telusdcinco commented 1 year ago

Migration Notes:

Run "php artisan migrate" to update tables Run "php artisan db:seed --class=DataFix_1138_Setup_Excused_Departments" to update 5 dept IDs to be excluded. Run "php artisan db:seed --class=DataFix_1138_Add_Excused_Reason_Department" to update Departments into Excused Reasons. Run "php artisan command:CalcNextConversationDate" to update excused statuses. Run "php artisan command:PopulateUsersAnnexTable" to update static details.

The code changes for #1138 may touch on related code modified by #1142.

telusdcinco commented 1 year ago

Also fixed a bug on the modal page, wherein the Reason displays "PeopleSoft Status" for "Classification" rows.

telusdcinco commented 1 year ago

Private Zenhub Image

telusdcinco commented 1 year ago

@steffipravasi Ready for retest in Test.

steffipravasi commented 1 year ago

Test data - Dept id: 039-8005 and 112-0186 Once the process runs, the employees within these departments are excused. But anyhow can still receive email notifications. Assigning ticket to Dennis to block the notifications (emails) for the excused employees. Email notifications can be received from:

steffipravasi commented 1 year ago

Test validated. Test data used for dept ids: 039-8005 and 112-0186 The department when excused are displayed with the properties: Excused=Auto, Reason=Dept ID, Excused By=System Employees also excused from email notifications.

Travis-A-Clark commented 1 year ago

@telusdcinco I just heard from the HR lead over at Attorney General. They want to reduce the number of Dept IDs that are auto-excused. I am assuming it is easy enough now to just edit the table if we need to add or delete?

To start, they only want to auto-excuse:

105-1115 105-1200

Can you let me know when the table has been updated? Thanks!

telusdcinco commented 1 year ago

Private Zenhub Image