bcgov / entity

ServiceBC Registry Team working on Legal Entities
Apache License 2.0
23 stars 57 forks source link

UI Design - Discharge #7561

Closed Kaineatthelab closed 2 years ago

Kaineatthelab commented 3 years ago

Discharge Visual Design Comps (including Security Modal): https://projects.invisionapp.com/share/VH11K6AFRDTS#/screens?browse

Also See Design Ticket with all Reg Table Dropdown Menus/Actions: https://app.zenhub.com/workspaces/entity-5bf2f2164b5806bc2bf60531/issues/bcgov/entity/7619

Wireframes (rough design):

Static page views ~https://app.uxpin.com/edit/8887562#?id_page=140340701~

Interactive mockup used for sprint review July 27, 2021 ~https://preview.uxpin.com/1348a94271171ee8e283ecf65f50b9cb4d5f8af2#/pages/140603350/simulate/no-panels?mode=~


No fee for any discharge User needs to input debtor name to discharge - look at options before open or when open Single step - review and submit on single screen - the registration cannot be amended at the same time as it's discharged - check this with Court Order info... Notify user that Secured Parties receive a copy of the Discharge Verification Statement. Show icon beside discharge menu item to indicate "password" required

Read only information - displays current state of the registration Length Trust Indenture (if present) Registering Parties Secured Parties Debtors Collateral General Collateral

Inputs for this registration Court Order section/information - TBD Folio/Reference number field for tracking

jinghualicgi commented 2 years ago

Don't have enough requirements now. A debtor name is required

janisrogers commented 2 years ago

Requirements https://docs.google.com/document/d/1DHPmLp1nH368anJX4kPntPlMhvo-74D-cPOoOpGvJLw/edit

yuisotozaki commented 2 years ago

@janisrogers to provide "Court Order" field set.

janisrogers commented 2 years ago

Court Name – 256 characters, Required Registry – 64 characters, Required File Number – 20 characters, Required Date of Order – Date, Required Effect of Order – 1000 characters, Required

yuisotozaki commented 2 years ago

@janisrogers @tlebedovich UXPin mock ready for your review.

Change Summary

General

Discharge

Renew

Amend

tlebedovich commented 2 years ago

@yuisotozaki @janisrogers

Notes from assets grooming:

Regarding required information before allowing a discharge:

From Patrick

From Kevin

Regarding Not charging for Discharge/Renew/Amend

From Kevin

yuisotozaki commented 2 years ago

Will add debtor name field to step 2. What happens if the debtor's name is "Sid Meier"? Is the first 5 letters "SIDME"?

janisrogers commented 2 years ago

@yuisotozaki Doug say the API requires an exact match of the full debtor name.

yuisotozaki commented 2 years ago

@janisrogers Then the rule is "Exact match including case with one of the debtor names on the registration."

yuisotozaki commented 2 years ago

☝️ Not sure if the API is doing what the requirement is asking for. The requirement says first 5 letters. 🤔

tlebedovich commented 2 years ago

@PatrickAHeath @Kaineatthelab - For the Security Step for a Total Discharge/Renew - Can you confirm whether we should use Debtor First and Last Name (which apparently is what the API is using), or First 5 letters of a debtor's name (what the requirements doc says).

Yui also mentioned - If the latter, what happens if the debtor's name is "Sid Meier"? Is the first 5 letters "SIDME"?

I think First and Last name is more straight forward for the user, if this could work with the regulations and requirements.

c/c @doug-lovett @jordiwes @yuisotozaki @janisrogers

yuisotozaki commented 2 years ago

Answer to above on RocketChat.

Details to still hammer out

tlebedovich commented 2 years ago

Conversation from Rocket Chat;

Yui: In both Discharge and Renew epics, there is a “you need to enter 5 letters of the debtor’s last name or business name” called out in the requirement documents. I heard that the API requires exact match (case sensitive?) for the full name of a debtor on the registration. So which is it? Just 5 letters or exact match? And for exact match, is it just for debtor’s name or can it be business name?

Patrick: The current PPR application requires the user to enter the first 5 characters of the debtor Last Name or Business Name, as well as the Base Registration Number. If the 5 characters entered are incorrectly, the system displays the correct 5 characters, which the user can then enter into the field and proceed with the renewal or discharge. The regulations indicate that the full debtor name is required - ‘the name of the base debtor exactly as entered on the base registration’, which was what Kevin brought up last week when we were discussing this.

My understanding from Bob is that entering the 5 characters is another check that the user has the correct base registration number, since no other information is displayed about the registration before the renewal/discharge is completed. In the design for the new application, the information about the registration is displayed not only in the My Registrations list that they select from, but also within the screens for the activity. The user can see more information about the registration and easily tell if the base registration number they entered is the one they want to act on. It seems silly to require a user to enter the first 5 characters of the debtor name that is displayed in full on the same or prior screen.

Kaine: I believe that is what we decided in that last meeting. So if the registration is in your dashboard you can do those actions and the end modal makes you double check…. When adding a registration to your dashboard you do the 5 letter modal

the api is currently exact match but it should work exactly like old system

Yui: I think we landed, on Friday last week, on having the debtor verification at the beginning of the Discharge/Renew flow so that we don’t have users putting in the effort only to find that they had the wrong registration open at the end.

Patrick: If the registration is already in the My Registrations list on the dashboard, the user will already see registration details and not have to enter the first 5 characters of the debtor.

I think having an additional check for debtor name would be useful when the user requests a registration be added to their My Registrations list, but once there and they can see the details of those registrations, allow them to select the registration to act on. A ‘Are you sure you want to do this?’ modal should be included on each activity as a last chance exit for the user.

Yui: For that last chance exit, sounds like the current regulation requires the new application to continue to have a debtor name verification modal for discharge and renew. Once the regulation changes, we can replace these verification modals with a simple “Are you sure you want to do this?” modal.

tlebedovich commented 2 years ago

image.png

tlebedovich commented 2 years ago

FINAL DECISION (FOR NOW):


tlebedovich commented 2 years ago

@yuisotozaki - comment from sprint review:

tlebedovich commented 2 years ago

@yuisotozaki - if your wiresframes are done for discharge and renew, you can move the tickets back to in progress and they'll go to my design to do list and you can assign them to me.

yuisotozaki commented 2 years ago

Back to In Progress for visual design.

yuisotozaki commented 2 years ago

Policy note around "Expiry Date" https://github.com/bcgov/entity/issues/7560#issuecomment-890207580

tlebedovich commented 2 years ago

notes: dashboard modal updates -

yuisotozaki commented 2 years ago

UXPin updated

tlebedovich commented 2 years ago

@yuisotozaki @janisrogers - my notes from the call --

Add a statement to both discharge and renew letting users know that the registration may have been subject to an amendment/court order(s) and that this is just showing a current snapshot of the registration, there may be additional information then what is displayed (users can do a search to see the full history of this registration.

Kaine mentioned writing two versions of the statement - one general and one specific if we can know for sure.

please add more details if I misunderstood

it can be in regular text, a yellow alert is not needed.

yuisotozaki commented 2 years ago

UXPin updated

janisrogers commented 2 years ago

@yuisotozaki I don't think "subject to an amendment" makes sense as the text. We should discuss.

yuisotozaki commented 2 years ago

@janisrogers I could use some technical lingo tutelage for sure. I was just looking up what I heard during the meeting.

yuisotozaki commented 2 years ago

Asset Grooming Meeting Decisions

PatrickAHeath commented 2 years ago

@yuisotozaki For the wording related to performing a search to retrieve complete registration details, here's my suggestion:

Other activity exists for this registration. To obtain complete details, it will be necessary to conduct a Registration Number search.

If the only registration entry for this Base Registration number is the original registration (no amendments or other activity), the complete registration is already displayed and there's no need for the message.

Also, performing a Registration Number search can only result in one exact match (or a nil report if they enter the number wrong). If they perform any other search, they may end up with multiple results and there could be confusion as to which registration they should be looking at.

yuisotozaki commented 2 years ago

@tlebedovich ☝🏻 Additional recommendations from Patrick re: text content.

PatrickAHeath commented 2 years ago

Decision from August 16 standup: (revised again) Debtor Verification which looks at Base Debtor Last Name or Business Name will be modified as follows:

The term Base Debtor will not be added to the Search output PDF.

tlebedovich commented 2 years ago

Thanks @PatrickAHeath - I think we were going to have the user put it the full debtor's last name or full business name, just for ease of use, but the check will just be for the first 5 characters. And I'm hoping if its not a match then we can show the full last name or full business name for users as the hint (versus just first 5). Just a technical FYI.

PatrickAHeath commented 2 years ago

@tlebedovich Yes - the magic behind the curtain!

yuisotozaki commented 2 years ago
janisrogers commented 2 years ago

@PatrickAHeath I think the decision might have been to allow entry of any debtor, not just any active debtor, because we want to support people entering the base debtor even if it has been removed.

PatrickAHeath commented 2 years ago

@janisrogers I heard Active Debtor, which is why it's worded that way. I support your reasoning.

janisrogers commented 2 years ago

I think the first active debtor was what we will show if they get it wrong, but they can enter any debtor.

Kaineatthelab commented 2 years ago

yes

yuisotozaki commented 2 years ago

August 18 Assets stand-up discussion minutes around the "Confirm registration to be discharged" modal. EDIT: BA/Design Grooming clarification included from August 18

@tlebedovich @forgeuxGH5 Please review above list to assist Kial with dev.

tlebedovich commented 2 years ago

@forgeuxGH5 - I'm not sure that is how type-ahead works - ie Bullet 2 above about keeping the full list of debtor names even if they have typed some letters that aren't matching, otherwise it ends up more like an open text field, but we don't want the user typing their own name in if its not included in the list. The typehead works by filtering through a restricted list.

janisrogers commented 2 years ago

Going with stock vuetify auto-complete component as in Tracey's design.

jinghualicgi commented 2 years ago

@tlebedovich Hi Tracey, I am going to closed Done ticket after tickets stay there more than 2 days. I saw you reopened some UI Design ticket, Please let me know if you have any suggestions regarding this process. Thanks

tlebedovich commented 2 years ago

hi there

Sometimes I will finish/close a design ticket but then a new dev ticket needs it so I have to reopen my design ticket to be able to link it to the new dev ticket. its a flaw of zen hub unfortunately.

jinghualicgi commented 2 years ago

Thanks Tracey, so I will continue close tickets by following 2 day rule, please feel free to reopen design ticket when needed. Thanks