OSE - New Processes For Neighborhood Block Party Folder (TL)

atdservicebot commented 4 years ago

As an Street Event Reviewer, I need to track department specific reviews for on RW, Work Type Neighborhood Block Party. Those department reviews are APD Review, Fire Review, and Trash and Recycling Review. These reviews need to populate on those department's user group tasks lists.

Requester: Maria Alonzo DTS Manager: Tracy CTM Help Desk Ticket # : 1969325

Acceptance Criteria:

Previous Acceptance Criteria:

On folder type RW, Street Event, and work type Neighborhood (NPH), the folder will insert 3 review processes when an Approved attempt is saved on the Event Review process (50560)
- 72014 APD Review, 50040 Fire Review, 72008 Trash and Recycling Review.
The "Approved" attempt should not close out the Event Review process, it should stay open
The 3 processes should be assigned to the correct groups
- ACE APD, ACE ARR Review, Fire Review
The 3 processes are grouped by 'Review' type
Should there be a calculated end date? Yes and it should be the Event Start date
The 3 New Review processes will automatically close (if still open) when certain attempts are saved on any of the 3 New Review processes (ACE APD, ACE ARR Review, Fire Review)
- Attempts: Approved, Rejected, Withdrawn
Once any of the 3 New Review Processes have an attempt of Approved, Rejected, Withdrawn; It will close out all 3 New Review Processes as well as the Event Review Process with a status of "Closed"

"Event Intake" does not be inserted at all, "Event intake" Process needs to be not be inserted automatically "Event Review" Process needs to already be there when the folder is created The Trash and Recycling Review need to be assigned to ACE ARR Review group The APD Review need to be assigned to ACE APD Reviewer On the Info tab: Add "Neighborhood Block Party" to Event type info

The "Approved" attempt should not close out the Event Review process, should stay open Should there be a calculated end date? Yes and it should be the Event Start date Once any of the 3 New Review Processes have an attempt of Approved, Rejected, Withdrawn; It will close out all 3 New Review Processes as well as the Event Review Process with a status of Closed

Next Steps

LIT Development

Request Details HD 1969325.pdf

Migrated from atd-amanda #112

TracyLinder commented 4 years ago

Neighborhood block parties do not have to go through the bigger ACE process, just an OSE process. There is already a work type for neighborhood block party, the processes just need to be updated. The three processes requested are for other department review:

APD, FIRE, Trash & Recycling

TracyLinder commented 4 years ago

From Maria:

72014 APD Review 50040 Fire Review
72008 Trash and Recycling Review

TracyLinder commented 4 years ago

Nate will work on getting this backlog ready by Sprint 26.

TracyLinder commented 4 years ago

Status: Nate to perform analysis during sprint 26.

Nadin-Nader commented 4 years ago

Hi Maria,

Sorry for the second email but I also wanted to include #1709 OSE New Process for NPH. We can have a quick conversation on this one to get it started as it may require lots of details to go over a new process.

Please let me know what day works for you from my previous email (tomorrow Tuesday 6/9/2020 from 1-2pm or 3-4pm and I also have Wednesday 6/10/2020 from 10-11am or 1-2pm)

Respectfully,

Nadin A. Nader IT Business Systems Analyst Data and Technology Services Austin Transportation Department

From: Nader, Nadin Sent: Monday, June 8, 2020 2:57 PM To: Alonso, Maria Maria.Alonso@austintexas.gov Subject: #2662 RW, OSE - Info Tab Cleanup and #2664 OSE Folder Status Bug

Hi Maria,

Hope you are doing well. I am reaching out to you regarding #2662 RW, OSE - Info Tab Cleanup and #2664 OSE Folder Status Bug. I was wondering if we could set up a meeting for a quick overview on what is going on and what the goal of the request is. I also want to go ahead and define the acceptance criteria for the requests that we have so far.

I have time tomorrow Tuesday 6/9/2020 from 1-2pm or 3-4pm and I also have Wednesday 6/10/2020 from 10-11am or 1-2pm. Please let me know what time works best for you and we can schedule a call.

Once we have the information for this, we will be able to move this along and start getting it looked at further or worked.

Respectfully,

Nadin A. Nader IT Business Systems Analyst Data and Technology Services Austin Transportation Department

Nadin-Nader commented 4 years ago

Need the new Processes to be added when they NPH Work Type is used.

OSE gets the folder RW SUB= Street Event (SE) in Internet Pending, The user processes it and "Accepts" the application and then they add the work Type of Neighborhood. This is when they would like the new processes to be added. From here they should show up in the OSE office of APD task list from here they can review it. Right now they are doing this process manually.

Maria would like the new workflow of the processes to function the same as the RW, SE folder when they receive the folder and accept/approve.

Process to be added:

72014 APD Review 50040 Fire Review 72008 Trash and Recycling Review

Meeting Link: https://web.microsoftstream.com/video/d880b737-987b-47d3-a50a-26e3f57eb549

Next Steps

See if LIT can do
Set up meeting Between LIT and Maria to fine tune requirements

TracyLinder commented 4 years ago

We will finish defining requirements for this in Sprint 27.

Nadin-Nader commented 4 years ago

From Card #3052: From the meeting it looks like the requirements that we have so far may be enough for LIT to start researching and working on this.

When the work Type Neighborhood (NPH) is used on the RW, SE folder and the reviewer Accepts/Approves, three new processes should be added. 72014 APD Review, 50040 Fire Review, 72008 Trash and Recycling Review. Once Accepted or Approved, The RW, SE Folder with NPH Work Type should show up in the OSE office of APD tasklist. (This Process is currently being done manually). The new processes should make their workflow and the folder to function the same as the RW, SE folder when they receive the folder and accept/approve. Waiting on Sree to look and verify that it is enough info to start.

Nadin-Nader commented on Jun 29, 2020 Sree got back and said that this is do able but needed to know if this was applicable to Web Applicants or if they can use the one process that is shared "Event Intake". If not then the Web Acceptance process would have to be added and adjusted to others. Sree will work on this and will get back to us if anything on #1709.

New acceptance criteria summary has been added to #1709: Description Of User's Issue Add 3 default folder processes. The processes should insert when an attempt result of 'Accepted' is saved on the Event Intake folder process.

Acceptance Criteria

When the work Type Neighborhood (NPH) is used on the RW, SE folder and the reviewer Accepts/Approves on Event Intake process, three new processes should be added. 72014 APD Review, 50040 Fire Review, 72008 Trash and Recycling Review. The 3 processes should be assigned to the correct groups; APD Review, Trash and Recycling Review, Fire Review The new processes should make their workflow and the folder to function the same as the RW, SE folder when they receive the folder and accept/approve. Next Steps

LIT Development

TracyLinder commented 4 years ago

I just updated the issue description & added more acceptance criteria. One is TBD - should there be a calculated end date on the 3 new processes?

I'll send Maria an email asking for clarification

TracyLinder commented 4 years ago

Email to Maria:

Hi Maria,

I have more questions about #1709, on the 3 new processes being added: • Should we add an auto-calculated process end date? o This would inform the reviewers of how long they have to submit a review. • In cases where the reviewer never saves an attempt on these processes, it will cause the process to remain open on their task list. To remedy this, I’m suggesting that any remaining open processes be automatically closed by the system when you complete the Event Review Process with either approved, rejected, or withdrawn. Will this work for you?

Thanks for your help with getting this defined.

All the Best,

Tracy Linder, Product Manager Data & Technology Services 🚲 Austin Transportation ————————————————————————

Sreelatha-BlazeKannan commented 4 years ago

Reply from Maria on 06/30/2020 -

I have more questions about #1709, on the 3 new processes being added: • Should we add an auto-calculated process end date? No, thank you. They are more of a curtesy review rather than an official one. o This would inform the reviewers of how long they have to submit a review. • In cases where the reviewer never saves an attempt on these processes, it will cause the process to remain open on their task list. To remedy this, I’m suggesting that any remaining open processes be automatically closed by the system when you complete the Event Review Process with either approved, rejected, or withdrawn. Will this work for you? That sounds perfect! Thank you!

Answers are in BOLD

TracyLinder commented 4 years ago

Status: LIT will begin development during sprint 28.

TracyLinder commented 4 years ago

Status: LIT has begun development. Updating to sprint 29.

Sreelatha-BlazeKannan commented 4 years ago

Demo Sent on 07/14/2020 and waiting for feedback

Nadin-Nader commented 4 years ago

@Sreelatha-BlazeKannan @aishwarya2505 @TracyLinder Demo looks good and seems like the requirements were met. Tracy, should we get this on DEV and have a demo with the stakeholders when its on DEV?

TracyLinder commented 4 years ago

@Nadin-Nader That sounds like a great plan. @Sreelatha-BlazeKannan this is ready for DEV.

Approved for DEV

Sreelatha-BlazeKannan commented 4 years ago

@TracyLinder , @Nadin-Nader - Aishwarya is working on this, however has conflict on the commit #2391 after completing that she will commit #1709 changes to DEV

Nadin-Nader commented 4 years ago

@Sreelatha-BlazeKannan @aishwarya2505 @TracyLinder

Request to move to Dev has been sent to CTM DevOps team via SCTASK0087448 Also asked that this be deployed after SCTASK0087433 #2391

I also had to split this deployment as there was a request to have a Non-User To-Do account created. That will have to be created by CTM Tango and the CM Team. The task for the Non-User To-Do account is SCTASK0087458

Nadin-Nader commented 4 years ago

@Sreelatha-BlazeKannan @TracyLinder @aishwarya2505 Per Martin This (#1709 SCTASK0087448) has been deployed along with #2391 SCTASK0087433

TracyLinder commented 4 years ago

Status: Pending a portion of work to be promoted to DEV, then Nate or Tracy will test.

Nadin-Nader commented 4 years ago

@TracyLinder @Sreelatha-BlazeKannan @aishwarya2505 Gita has created the account and closed the TASK. her informational comment was: GR Gita Rajagopalan Work notes•08-05-2020 12:41:35 Done on DEV .

Such accounts do not have a database account and exist as records in the ValidUser table with group of "Group Account" and Department of "Non-User ToDo Account"

Nadin-Nader commented 4 years ago

STATUS: On DEV pending UAT

aishwarya2505 commented 4 years ago

@Nadin-Nader , I shall test it now in DEV

aishwarya2505 commented 4 years ago

@TracyLinder @Sreelatha-BlazeKannan @Nadin-Nader Tested 11884719 (RW , Street Event, Neighborhood) and 11884720 (RW, Parking Spaces, Neighbourhood). The validations work only for Street Event and Neighborhood.

Nadin-Nader commented 4 years ago

@aishwarya2505 @Sreelatha-BlazeKannan Commit b205878910e6 is still in their to do column (SCTASK0089966). Gita just completed the Account portion of the work.

Nadin-Nader commented 4 years ago

@aishwarya2505 @Sreelatha-BlazeKannan @TracyLinder Commit b205878910e6 (SCTASK0089966) Has been completed and deployed. All portions of the work should be deployed and ready for Testing

aishwarya2505 commented 4 years ago

@Nadin-Nader @TracyLinder @Sreelatha-BlazeKannan I am currently working with Martin as I am unable to log in to DEV. I should be able to test post resolving it.

aishwarya2505 commented 4 years ago

@TracyLinder @Nadin-Nader @Sreelatha-BlazeKannan tested this and it works as expected. (11889876, 11873936)Please go ahead and test this one.

Sreelatha-BlazeKannan commented 4 years ago

@TracyLinder , @Nadin-Nader - DEV Testing completed by Aishwarya and Ready for DEV UAT

Nadin-Nader commented 4 years ago

Status: Currently Testing in DEV

Nadin-Nader commented 4 years ago

@Sreelatha-BlazeKannan @aishwarya2505 @TracyLinder I was able to test this on DEV (Row ID 11895073) and all 3 review processes are inserted and the 3 processes close when the Event intake Process has an attempt of Approved, Rejected or Withdrawn.

I approve to move to TEST

aishwarya2505 commented 4 years ago

Test Deployment plan sent - 9/17/2020

Nadin-Nader commented 4 years ago

@Sreelatha-BlazeKannan @aishwarya2505 @TracyLinder I have submitted the CM DevOps Team request to get this on TEST. The task for this is SCTASK0096301

Nadin-Nader commented 4 years ago

@aishwarya2505 @Sreelatha-BlazeKannan @TracyLinder CTM DevOps Team has deployed this to TEST and closed out SCTASK0096301

Nadin-Nader commented 4 years ago

**Next Steps:*** UAT in TEST

Nadin-Nader commented 4 years ago

@Sreelatha-BlazeKannan I have UATd this on TEST and it looks good. All the acceptance criteria has been met. I have sent a request to Maria so she can UAT it with me and approve the move to PROD.

Next Step: Maria to UAT and approve. Waiting on meeting acceptance for today 10/1/2020

Nadin-Nader commented 4 years ago

@Sreelatha-BlazeKannan @aishwarya2505 @TracyLinder Spoke to Maria and she found something that needs a change. She mistakenly said that she wanted the 3 processes added to the Event Intake Instead of the Event Review.

She actually wants the 3 new processes to be inserted when the "Event Review" Process (50560) is Approved and NOT when the "Event Intake" Process (50555) is Approved.

I asked her the following:

"Does the Event Review process get added when the Intake is approved, do you want it added when the folder is created or info selected or do you want it to be added manually?"

I set a time to go over my last question and give her time to test it more. Meeting set for 10/2/2020 @930am

Nadin-Nader commented 4 years ago

Updated Acceptance criteria to reflect the 3 new processes to be inserted when the "Event Review" Process (50560) is Approved and NOT when the "Event Intake" Process (50555) is Approved.

Nadin-Nader commented 4 years ago

@Sreelatha-BlazeKannan @aishwarya2505 I spoke to Maria and Laura and had them show me what changes are needed. The changes are:

They will need to have Event Intake not be inserted at all, Event intake needs to be taken out Event review Process needs to already be there when the folder is created The Trash and Recycling Review need to be assigned to ACE ARR Review group The APD Review need to be assigned to ACE APD Reviewer

On the Info tab: Add "Neighborhood Block Party" to Event type info

Please let me know if you need clarification on this

Updated Acceptance criteria

Nadin-Nader commented 4 years ago

Email sent to Aish in order for work to resume:

Aish,

Hope you are doing well. So what they now want is, to have the three new review processes for Fire, APD and Trash and Recycling (Fire Review, ACE APD and ACE ARR Review) inserted into the “Event Review“ Process but they have always wanted the 3 new processes inserted when an attempt of “Approved” is entered on the Event Review Process.

The Event Intake process should be taken out all together and the Event Review process should be there when the folder is created. The Event Review process should take the place of the Event Intake process and do what the Event Intake process did

Does this answer your question?

aishwarya2505 commented 4 years ago

@Nadin-Nader Thank you Nadin. Yes, you got them all :)

Sreelatha-BlazeKannan commented 4 years ago

@TracyLinder , @Nadin-Nader - Aishwarya will work on the updated Acceptance criteria.

Nadin-Nader commented 4 years ago

@aishwarya2505 @Sreelatha-BlazeKannan I have updated the Acceptance criteria with the following to answer your question from the demo:

The 3 New Review processes will automatically close (if still open) when certain attempts are saved on any of the 3 New Review processes (ACE APD, ACE ARR Review, Fire Review) Attempts: Approved, Rejected, Withdrawn

Let me know if it makes sense now

@aishwarya2505 I have also reviewed the demo and found that everything looks good minus what you were asking

Nadin-Nader commented 4 years ago

@aishwarya2505 @Sreelatha-BlazeKannan @TracyLinder i updated the Acceptance criteria again but to answer your questions from the demo. Here is what i added, let me know if it makes sense:

The "Approved" attempt should not close out the Event Review process, should stay open

Should there be a calculated end date? Yes and it should be the Event Start date

Once any of the 3 New Review Processes have an attempt of Approved, Rejected, Withdrawn; It will close out all 3 New Review Processes as well as the Event Review Process with a status of Closed

aishwarya2505 commented 4 years ago

Thank you @Nadin-Nader. I shall make these fixes.

Sreelatha-BlazeKannan commented 4 years ago

@TracyLinder, @Nadin-Nader - Aishwarya will be working on #1709 Changes.

Sreelatha-BlazeKannan commented 4 years ago

@TracyLinder, @Nadin-Nader - Aishwarya working on the Commit changes will update by EOD.

Sreelatha-BlazeKannan commented 4 years ago

@Nadin-Nader @TracyLinder - DEV Commit submitted on 10/21/2020

Nadin-Nader commented 4 years ago

@Sreelatha-BlazeKannan @aishwarya2505 @TracyLinder I have sent this and the other commit (cf20482c8edd) to the CM Team for deployment to DEV under TASK SCTASK0101770

Nadin-Nader commented 3 years ago

@Sreelatha-BlazeKannan @aishwarya2505 @TracyLinder this has been deployed to DEV.

Status: Ready for DEV UAT

cityofaustin / atd-data-tech