bcgov / cas-cif

CleanBC Industry Fund project
Apache License 2.0
6 stars 2 forks source link

As a user, I want to be able to edit in-progress amendments/revisions via the Forms Updated section, so that I can input changes #1284

Closed BCerki closed 1 year ago

BCerki commented 1 year ago

Description:

NOTE: This ticket is outdated, check wireframes before refining. A tasklist-based editing workflow is here:https://github.com/bcgov/cas-cif/issues/1561

In this ticket, we change the workflow for editing amendments/general and minor revisions. This first step is to remove the "edit" button from forms. As a result, the tasklist can only be used to view forms. It can no longer be used to access or navigate editing functionality (with the exception of routinely updated fields, which always appear on the view page).

To edit amendments/general and minor revisions, a user must navigate to the desired project in the holistic change table and click the "View/Edit" button. After clicking the button, the user arrives at this page: image image

To access an in-progress form to make changes, users must click on the form's title in the "Forms Updated" section, which links to the form. The fields that don't apply to a particular revision type should be greyed out, but otherwise the form is editable.

If the user clicks on the tasklist at any point while working on an in-progress form, they're ejected from edit mode and sent to view the latest committed revision of that form. To get back to editing that form, they have to refind the project in the holistic change log and click "View/Edit" and the form's title again.

Outstanding questions:

Old context:

**After making a revision of any type, the "Amendments & Other Revisions" tab in the task list disappears. To continue with updates or check that the changes were captured, a user has to exit the project, then re-find it in the project list, which is inconvenient.

Keeping in mind the desired functionality in these issues: -https://app.zenhub.com/workspaces/climate-action-secretariat-60ca4121764d710011481ca2/issues/gh/bcgov/cas-cif/1119 -https://app.zenhub.com/workspaces/climate-action-secretariat-60ca4121764d710011481ca2/issues/gh/bcgov/cas-cif/1122

Envisioned workflow (prototype)**

Acceptance Criteria:

Given I am a user and I navigate to any of the forms using the tasklist When I view the form Then I do not see an edit button

Given I am a user and I navigate away from a revision and return to it (by clicking "View/Edit") When I click on any of the form titles in the "Forms Updated" section Then I am taken to the form And the fields that cannot be updated via a revision of amendment type are greyed out And beside those fields is an explanation for why they are greyed out

Given I am a user and I look at the tasklist When I click any of the links I am ejected from edit and sent to view And I see the appropriate routinely updated fields

Development Checklist:

Definition of Ready (Note: If any of these points are not applicable, mark N/A)

·Definition of Done (Note: If any of these points are not applicable, mark N/A)

Notes:

-

LindsayMacfarlane commented 1 year ago

Hi @nanyangpro @suhafa, Is there any existing design work that supports this ticket? Happy to walk through and discuss. This may be a moot issue with ticket #1158.

nanyangpro commented 1 year ago

Thanks for confirming, @LindsayMacfarlane! Yes, we may already have the user story covered in one of the existing prototypes. I just need a bit of clarification/context of the ticket itself, and would appreciate a quick chat before dropping the link. Would you recommend me reach out to @BCerki directly or call you instead?

LindsayMacfarlane commented 1 year ago

@nanyangpro - you can reach out to me.

nanyangpro commented 1 year ago

Wonderful, thanks Lindsay! If you are available, I'll call you after our Dev/Design Sync today (with a 5-min break for both of us :).

nanyangpro commented 1 year ago

Appreciate for the walkthrough, @LindsayMacfarlane! As discussed, the link to the envisioned workflow (prototype) has been dropped in the ticket description. Thank you!

BCerki commented 1 year ago

@nanyangpro , where does the "New Revision" link go? And is the link text always "New Revision" (vs. Add, Edit, etc.)?

nanyangpro commented 1 year ago

Thanks for your question, @BCerki!

BCerki commented 1 year ago

Thanks, @nanyangpro ! A couple follow-up questions:

  1. do we send the user to the "New Revision" page even if they're returning to a revision they've already created? (Ie, the already chose the forms to be updated and already clicked the "new revision" button).
  2. how does the user get to the page with pending actions from and the revision record history?
BCerki commented 1 year ago

To clarify the questions, when the user clicks the "New Revision" link in the tasklist: image They should be redirected where? I'm guessing it's one of these two pages, or possibly conditional depending on what stage the revision is at:

Page 1 image (should a user ever see this page except for when they're creating the revision for the first time?)

Page 2 image (currently the only way to get to page 2 is to exit the revision and then come back into it through the "View/Edit" button in the table, see Loom with audio)

Sidenote, this is what currently happens if you click the "New Revision" button if there's already a revision: https://www.loom.com/share/532da8abe2e24ce38c170db6c658d05e

@Sepehr-Sobhani , @gurjmatharu , we've all had this workflow come up in PRs lately; feel free to add to or edit this comment.

Sepehr-Sobhani commented 1 year ago

Just some context and sharing my thoughts :) We don't create a new revision/amendment before hitting New Revision button, and after hitting the button we are in the edit mode and there's an ongoing revision so maybe the most straightforward way of having access to the revisions/amendments table is to bringing it back in the tasklist or maybe having a button that takes the user directly to the revision/amendment edit/view page.

gurjmatharu commented 1 year ago

Just some context and sharing my thoughts :) We don't create a new revision/amendment before hitting New Revision button, and after hitting the button we are in the edit mode and there's an ongoing revision so maybe the most straightforward way of having access to the revisions/amendments table is to bringing it back in the tasklist or maybe having a button that takes the user directly to the revision/amendment edit/view page.

Also when you're editing a revision you can't edit another revision so even if you have access to the list you won't be able to switch editing revisions.

BCerki commented 1 year ago

Just some context and sharing my thoughts :) We don't create a new revision/amendment before hitting New Revision button, and after hitting the button we are in the edit mode and there's an ongoing revision so maybe the most straightforward way of having access to the revisions/amendments table is to bringing it back in the tasklist or maybe having a button that takes the user directly to the revision/amendment edit/view page.

To make sure I've got this straight since there are multiple New Revision buttons: -the button on the Amendments and Other Revisions page above the table is just a redirect; it doesn't actually create the revision

So I think that means the user can never get back to page 1 once they've created a revision. (Similar to how when you first create a project you get to the select rfp form, and once you've clicked create project, you can never get back to the select rfp form again.) So that means that after clicking New Revision on page 1, the user can never change amendment type general/minor/amendment or scope/cost/schedule?

Sepehr-Sobhani commented 1 year ago
BCerki commented 1 year ago

Updated the ticket title and AC. @nanyangpro will create a ticket that will block this one

BCerki commented 1 year ago

This could be split into separate tickets for each form. A few more questions:

Dianadec commented 1 year ago

Next steps from Feb. 7 backlog refinement:

LindsayMacfarlane commented 1 year ago

Backlog refinement (Feb 22, 2022): There are still a lot of outstanding questions to support moving forward with amendments and revisions. Taking a pause to do a more in-depth review/whiteboarding session before continuing with the implementation of this EPIC.

pbastia commented 1 year ago

this might still be relevant after #1523 is completed. unless @nanyangpro do you have an answer for that?

pbastia commented 1 year ago

Not needed at this point, we'll create a new card with updated prototypes when needed.