cityofaustin/atd-data-tech

RW Unpaid Fee Flag (SS) (On PROD)

Closed this issue · 15 comments

Acceptance Criteria

  • System should check to see if any of the People records in the folder have unpaid fees when user makes the following attempts.

Folder Type SubType Process Attempt
RW Street Vendor Vendor Permitting Update Issue and Expiry Dates
RW Valet Operator Valet Operator Permitting Update Issue and Expiry Dates
RW Valet Parking Annual Annual Valet Permitting Update Issue and Expiry Dates
RW Temporary Right of Way Use Permit Permit Extension Manual Extension or Extension Granted

  • System should check to see if any of the People records in the folder have unpaid fees in any of the following folder types: RW, EX, LC, ECV with ATD subtype, GF with ATD subtype, or DS.
  • If there are unpaid fees, system must allow the attempt to complete but throw a warning message "A People record from this folder has UNPAID fees"

@SubhaShroff this has been sent tp the DevOps Cm Team to be deployed to DEV under SCTASK0422984

@Nadin-Nader RW - TURP - Permit Extension Process - Manual Extension attempt already has a warning message.
'Please make sure Applicant's Bond, Insurance, Fee and Date Range information are up to date.'
Now with this ticket, the requirement is to add - "A People record from this folder has UNPAID fees".

Can we club both messages as we can't display two warnings with one attempt?
If we can, any suggestion to the updated warning message wordings, how the users want it?
Thank you.

Conversation with Kyle about Subha's Question:

[Yesterday 2:47 PM] Nader, Nadin
Hi Kyle. Hope you are doing well. Hey do you have a sec for a question on 17028 (RW Unpaid Fee Flag)?
[Yesterday 2:48 PM] Nader, Nadin
RW - TURP - Permit Extension Process - Manual Extension attempt already has a warning message.
'Please make sure Applicant's Bond, Insurance, Fee and Date Range information are up to date.'
Now with this ticket, the requirement is to add - "A People record from this folder has UNPAID fees".

Can we combine both messages as we can't display two warnings with one attempt?
If we can, any suggestion to the combined warning message wordings what the users want it as?
[Yesterday 3:20 PM] Kampe, Kyle
I don't think we'd want to combine the messages into a single message, because then it would presumably display even in instances where only the existing warning message applies to a scenario but not the new text warning of unpaid fees. We might want to discuss this one more in tomorrow's meeting. I really thought it was possible to have an additional warning message display after okaying an initial one.

Subha mentioned that it is possible to have both of these messages on the same error popup, as well as having each individually depending on the circumstance. ROW is OK with going forward with this.

@SubhaShroff this has been sent to the DevOps CM Team to be deployed in DEV under SCTASK0425289

@SubhaShroff this 2nd Dev commit has been sent to the DevOps CM Team to be deployed in DEV under SCTASK0426059

17028 - 9550845f044e7d403-583e4dc9bb94a3f33f-8122bcaf2f48b0e79f10 TEST Deployment Plan.docx
@Nadin-Nader Thank you for creating DEV deployment request. I tested and working now as expected. Please create a TEST deployment request.

It has following 3 commits:

  1. 9550845f044e7d403067208f7b50331d2a5f862d
    Process_RW_00050680: PROCESS_RW_00050650: Process_RW_00050590: Process_RW_00050630: Add a warning message, if any associated people have unpaid fees

  2. 583e4dc9bb94a3f33f39990d077a4d07f13f4fec
    Process_RW_00050680: PROCESS_RW_00050650: Process_RW_00050590: Process_RW_00050630: Added a logic to check unpaid billed fees as well, to display the Warning message with the respective attempts in the above process procedures

  3. 8122bcaf2f48b0e79f100bb487ce23cd33829ae3
    PROCESS_RW_00050680.sql: Remove the error display at this point.

Thank you.

@Nadin-Nader SCTASK0427118 - TEST deployment request is created with Priya's help. Thank you.

@SubhaShroff this has been deployed to TEST please UAT

@Nadin-Nader I tested the deployments on TEST. This ticket is ready for UAT. Thank you.

@Nadin-Nader @SubhaShroff UAT on Test approved. Thank you!

This has been sent to the DevOps CM Team to be deployed to PROD under SCTASK0429097

@Nadin-Nader This is deployed in PROD, though Hari didn't update the status yet on SCTASK0429097. But he confirmed that its deployed.
This is ready for UAT on PROD.
Thank you.

@Nadin-Nader @SubhaShroff UAT on Prod approved. Thank you!