Add 'Void' as a Deficiency Status (TL)

[atdservicebot]

As a ROW traffic control inspector, when I add a deficiency in error I need to indicate that the deficiency status is VOID.

Customer: Ryan, Nathan C, Kim DTS Manager: Tracy Note: Per Gita in 2018, deleting a deficiency is not an option. Will have to use deficiency status. Ready for Development? Yes

Acceptance Criteria:

• On process 50685 Traffic Control Inspection, the user is able to update a deficiency status to 'VOID'
• Deficiencies with a VOID status do not populate on RW folder reports 50514, 50563, 50516, 50568
• Deficiencies with a VOID status do not populate on EX folder reports 50510, 50562, 50514, 50516, 50563
• Deficiencies with a VOID status do not populate on ECV folder reports 50516, 50568, 50514, 50563
• Deficiencies with a VOID status do not populate on DS folder reports 50514, 50563, 50516, 50568
• On 50514; Deficiency Corrected Date Field: The query should validate the deficiency status (FolderProcessDeficiency.SatusCode) instead of whatever it's currently validating. The field should populate based on the following:
  • If Status = Corrected, then field value = Complied Date (FolderProcessDeficiency.DateComplied)
    • If Complied Date is Null, then field value = Corrected; No Date
  • If Status = Deficient, then field value = Pending

---

Request Date: 2018-08-03 00:00:00

Request ID: DTS18-100385

Status: Backlog

Customer Priority:

Level of Effort: Small Project

DTS URL: https://atd.knack.com/dts#service-requests/view-issue-details/5b64c9f3fbbcf814ca4a0096

Migrated from atd-amanda #23

[amenity]

Nathan C. can be a point of contact for this.

[TracyLinder]

Email from Subha:

Hi Tracy/Nadin,

Reports for EX, 50510 and 50562 do not have deficiencies queried in any section. I think requirement doesn’t apply for these two reports. Please clarify.

Thanks, Subha

My Response:

Hi Subha,

Thanks for letting us know. You are correct and we can remove that from requirements.

Thanks,

Tracy Linder, Product Manager

[TracyLinder]

Update

• Subha sent us a demo on 6/24
• I responded with the below on 6/30

Hi Subha,

The demo looks great. My only feedback is actually a new requirement, see below.

• On report 50514; Deficiency Corrected Date Field: The query should validate the deficiency status (FolderProcessDeficiency.SatusCode) instead of whatever it's currently validating. The field should populate based on the following: o If Status = Corrected, then field value = Complied Date (FolderProcessDeficiency.DateComplied)  If Complied Date is Null, then field value = Corrected; No Date o If Status = Deficient, then field value = Pending o See attached picture #1 - I've illustrated what your report would look like with the validation changes.

Thanks so much!

Tracy Linder, Product Manager

[TracyLinder]

I just updated the issue description with additional requirements that were emailed to Subha.

[TracyLinder]

Status: Subha will be sending us a demo soon. If that looks good, move this to DEV.

[Sreelatha-BlazeKannan]

@TracyLinder @Nadin-Nader - Subha Sent Demo on 7/1/2020 and waiting for Feedback.

[TracyLinder]

@Sreelatha-BlazeKannan @SubhaShroff - The demo looks great, please promote to Test.

[Nadin-Nader]

@TracyLinder @Sreelatha-BlazeKannan @SubhaShroff Deployment to DEV was on card #3324 and deployment to TEST was on #3325. Both connected successfully.

Other half of deployment needed to go to DevOps so Subha submitted a TASK to CTM DEVOPS TEAM ServiceNow request (REQ0056179 - SCTASK0080257). Its at the top of their "To-Do"

Next Steps ServiceNow request (REQ0056179 - SCTASK0080257 Needs to be completed

Need UAT and approval for it to be promoted to PRDO

[Nadin-Nader]

@Sreelatha-BlazeKannan @SubhaShroff Deployment of form has been done on DEV and TEST. Commit to promote to DEV has been completed by CM Team. @SubhaShroff Please let me know when this is ready to go to TEST then PROD after UAT

[SubhaShroff]

@Nadin-Nader I tested it in Dev. Its ready for Dev Testing. Please let me know, so that I can submit a ServiceNow request for TEST deployment of the commit.

[Nadin-Nader]

@SubhaShroff this has already been approved to move to TEST by Tracy

[Nadin-Nader]

Deployment to TEST with CTM Commit: d8a5cad09dbb - Task : SCTASK0080938

Waiting on Deployment to TEST, UAT then PROD

Created card #3342 for TEST DevOps deployment

[Nadin-Nader]

@TracyLinder this is ready for testing on TEST. I am closing #3342 as it is the deployment for TEST. STATUS:

Ready for UAT on TEST then Deploy to PROD

[SubhaShroff]

@TracyLinder @Nadin-Nader @Sreelatha-BlazeKannan This is tested in TEST. 50514, 50516 are working as expected for EX, RW, DS & ECV. Email report 50563( thats 50514) and 50568(email report of 50516) couldn't be tested with my email-id due to recipient email id validations set in TEST. Otherwise, its ready for TEST testing.

[TracyLinder]

UAT Results

50514 Tests

Test 1

Result: Failed

• Folder ROW ID: 12344083
• There are 3 deficiencies on the folder, however 1 has no deficiency status so it is not populating on the report. If a deficiency status is null, we should treat it as if the status were Deficient. I will update the acceptance criteria.

Test 2

Result: Passed

• ROW ID: 12344083
• Notes: I added a deficiency, then voided it. It did not populate on the report, as expected. 👍

50516 Results

Test 1

Result: Failed

• ROW ID: 12344083
• Same issue as the failure above. The report does not populate deficiencies with no deficiency status. They should be treated the same as those with status of Deficient.
• Void deficiency did not populate, which was good.

[TracyLinder]

@SubhaShroff @Sreelatha-BlazeKannan

UAT for this failed. My testing notes are available in the above comment, and the acceptance criteria has been updated. See below image where the updated portions are highlighted, and please let me know if there are any questions.

[TracyLinder]

Feedback to be incorporated by LIT during sprint 30.

[SubhaShroff]

@TracyLinder Yesterday, I updated the reports 50514, 50516 as per the updated requirement in UAT comment. I sent you the reports output as pdf files to you in email. The pdfs are the result of the same folder (12344083) that you have tested in TEST instance. So, please review it and let me know your feedback.

[Sreelatha-BlazeKannan]

@TracyLinder , @Nadin-Nader Reports (50514 & 50516) updated and submitted for review on 07/28/2020

[TracyLinder]

Review Notes:

I reviewed the 2 reports provided by @SubhaShroff

• 50514: Pass/Approved
• 50516: Failed

50516 Feedback for Subha:

System Deficiency	Deficiency Status	On Report?	Report Status	Pass/Fail	Notes
Missing	Corrected	No	N/A	Pass	N/A
Imporperly Located	Deficient	Yes	Pending	Fail	Report status should be: Non-Compliant
Blocked	Void	No	N/A	Pass	N/A
No Permit	Null	Yes	Pending	Fail	Report status should be: Non-Compliant
In road during peak hours	Corrected	Yes	Non-Compliant	Fail	Deficiency should not populate report

[Sreelatha-BlazeKannan]

@TracyLinder , @Nadin-Nader - Subha completed the changes and sent an email to Tracy on 08/04/2020 and waiting for feedback.

[TracyLinder]

@Sreelatha-BlazeKannan @SubhaShroff the report you sent me on 8/4 looks great. Please promote this work to test.

[Nadin-Nader]

@Sreelatha-BlazeKannan @TracyLinder @SubhaShroff I have deployed 50510 this to DEV and TEST.

[Nadin-Nader]

@Sreelatha-BlazeKannan @SubhaShroff i have deployed Report 50514 to DEV and TEST

[SubhaShroff]

Thank you @Nadin-Nader. @TracyLinder @Nadin-Nader @Sreelatha-BlazeKannan I tested 50514, 50516( for the fixed UAT comments) in DEV and TEST for all four folders: EX, DS, RW & ECV. Its ready for your testing in DEV as well as TEST with updated reports.

[TracyLinder]

I just tested in TEST, and this looks great! I'm going to schedule some UAT with stakeholders before approving a promote. Thanks @SubhaShroff

[TracyLinder]

Meeting w/ Stakeholders scheduled for 8/18 @3pm.

[TracyLinder]

I tested this with the stakeholders today & they approved for prod.

Approved for production. 🎉

[SubhaShroff]

Thank you @TracyLinder. @TracyLinder @Nadin-Nader PROD Deployment requests are submitted: SCTASK0090973, SCTASK0090979

[SubhaShroff]

@TracyLinder @Nadin-Nader @Sreelatha-BlazeKannan PROD Deployment is done.

[TracyLinder]

Stakeholders have been informed; closing ticket.