Skip to content

batches: details page cleanup

Warren Gifford requested to merge kr/details-list-cleanup into main

Created by: courier-new

A couple other small cleanup items that I did as part of #33395 (closed).

  • Updated the verbiage of the empty list element for changeset lists
    • Changesets won't be created until the batch spec is executed and applied.
    • The link takes you to the page to edit the latest batch spec, specifically, not just to view it.
    • The spacing also looked kinda weird here to me so I adjusted a bit.
  • The batch change state pill on the details page didn't show the "DRAFT" state, if a batch change was a draft, because it was using a totally different component. I refactored to use the same badge component as on the batch changes list page.
  • Updated some storybook stories

Test plan

These component have storybook story coverage, and I expanded the coverage as part of this PR. This PR doesn't significantly modify existing logic.

App preview:

Check out the client app preview documentation to learn more.

Merge request reports

Loading