Skip to content
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

PIC-3145: Probation record orders restructure #1027

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

KiraNamida
Copy link
Contributor

@KiraNamida KiraNamida commented Jan 15, 2025

[PIC-3145](https://dsdmoj.atlassian.net/browse/PIC-3145)

The Probation record orders (active and previous) with Summary Cards/Lists. This will thus be the foundation for PIC-3146 and PIC-3147 when they are picked up.

New Summary Card Layout:
A few samples of the new layout

Example of a simple order with no additional details
image

Examples of an order with an additional status and details
image
image

Example of an order with multiple additional statuses and details
image

A series of orders one after the other
image

A sample of a Previous Orders with the initial first 5 displayed
image

Additional steps:

  • New cypress steps for interrogating summary cards.
  • Collect the case-summary views into a domain specific directory to collect them together a found a partials sub-directory to isolate more complicated sections rather than writing them into monolithic files.
  • A new testing util to write relative dates into cypress test statements as some of our wiremock data is itself relative.
  • Separate Summary Card and Summary List common components with the Summary Card using the Summary List itself, this is to prepare for suggested future uses of Summary Lists without the surrounding card.

… toggle minor change and Summary Card replacement. Awaiting UX approval before continuing.
@KiraNamida KiraNamida force-pushed the PIC-3145/probation-record-orders-restructure branch from d1bb66b to e1ee046 Compare January 15, 2025 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant