-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
As a CIF administrator, I want to see clear context of the changes in Amendments/General Revisions, so that I know what I am about to override #1569
Comments
Hey team! Please add your planning poker estimate with Zenhub @BCerki @dleard @gurjmatharu @mikevespi @pbastia @sam-warren @Sepehr-Sobhani @tmastrom |
|
@gurjmatharu I think for the deleted data we've just used an italic DELETED everywhere |
Thanks, @gurjmatharu! As discussed just now, if there are further clarifications you need, feel free to reach out again! |
Since this ticket isn't going to include the contextual help (?), we can probably roll that in with the tooltip implementation in https://app.zenhub.com/workspaces/climate-action-secretariat-60ca4121764d710011481ca2/issues/gh/bcgov/cas-cif/1573 |
The contextual help is the core of this ticket and should stay in the scope. #1573 is simply hiding them behind the tooltip icon. |
Contextual text will be broken out into another card. |
|
Thanks for the hard work @gurjmatharu ! This was a complex one.
See EP001 in -dev, I've opened an amendment, then committed a couple of revisions. the changes from the revisions don't show in the amendment diff |
Description:
As outcome of #1523 (with back-end work handled in #1528), the design of the diff section for an Amendment or General Revision is refined to provide clear context of the changes.
- Design: Sequential Scenario 1
- Design: Sequential Scenario 2
- Design: Sequential Scenario 3
- Design (Cleaner Version): preferred by users, nice-to-have, out of scope of this ticket, will be handled in #1573
In summary, for the diff section, we'll show the following fields under their forms with help/contextual texts:
Acceptance Criteria:
Given I am a user
When I edit non-calculated fields on a form and another revision hasn't been committed since I opened the revision
Then when I view the "Forms updated" section
Then the old value is in black and struckout
and if there was no old value (the field was blank) I see "Not entered"
Given I am a user
When I edit fields that affect calculated fields on a form and another revision hasn't been committed since I opened the revision
Then when I view the "Forms updated" section
Then the old value is in black and struckout
and if there was no old value (the field was blank) I see "Not calculated"
and I see the help text (see wireframe)
Given I am a user
When I edit fields on a form and another revision hasn't been committed since I opened the revision
Then when I view the "Forms updated" section
Then the new value is in red
and there is an arrow between the old and new values
Given I am a user
When I edit fields on a form and another revision has been committed since I opened my revision, and the revisions do not touch the same fields
Then when I view the "Forms updated" section
I see the changes from my revision
and I see the changes from the other revision
and I see help text (see wireframe) below the changes from the other revision
Given I am a user
When I edit fields on a form and another revision has been committed since I opened my revision, and the revisions do touch the same fields
Then when I view the "Forms updated" section
I see the old value when I started my revision, then the other revision's value, then my new edited value (with colours and arrows in wireframe)
and I see help text (see wireframe)
NOTE: This ticket doesn't cover the yellow/green diffs. That page will be removed by https://app.zenhub.com/workspaces/climate-action-secretariat-60ca4121764d710011481ca2/issues/gh/bcgov/cas-cif/1582
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:
The text was updated successfully, but these errors were encountered: