Skip to content

[MPDX-9186] Add request summary card to step 4 #1574

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.

Sign up for GitHub

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

Jump to bottom
Open
canac wants to merge 3 commits into
base: main
Choose a base branch
from
Open

[MPDX-9186] Add request summary card to step 4 #1574

canac wants to merge 3 commits into from
+53 −4

Conversation

@canac
Copy link
Contributor

@canac canac commented Jan 15, 2026 •
edited
Loading

Description

  • Add the request summary card to step 4 (it is also on step 3)
  • Add the button to make the calculations collapsible

MPDX-9186

Testing

  • Go to step 4 of a calculation
  • Check that the request summary card shows up
  • Check that the calculations aren't shown at first
  • Click the View Complete Calculation button
  • Check that the calculations are shown

Checklist:

  • I have given my PR a title with the format "MPDX-(JIRA#) (summary sentence max 80 chars)"
  • I have applied the appropriate labels (Add the label "Preview" to automatically create a preview environment)
  • I have run the Claude Code /pr-review command locally and fixed any relevant suggestions
  • I have requested a review from another person on the project
  • I have tested my changes in preview or in staging
  • I have cleaned up my commit history

@canac canac requested a review from wjames111 January 15, 2026 21:59
@canac canac self-assigned this Jan 15, 2026
@canac canac added the Preview Environment Add this label to create an Amplify Preview label Jan 15, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 15, 2026

Preview branch generated at https://mpdx-9186-request-summary-card.d3dytjb8adxkk5.amplifyapp.com

@github-actions
Copy link
Contributor

github-actions bot commented Jan 15, 2026 •
edited
Loading

Bundle sizes [mpdx-react]

Compared against 249d245

No significant changes found

wjames111
wjames111 reviewed Jan 16, 2026
View reviewed changes
Copy link
Contributor

@wjames111 wjames111 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@canac canac enabled auto-merge January 16, 2026 19:36
@canac canac force-pushed the mpdx-9186-request-summary-card branch from f530bec to 9f25ead Compare January 16, 2026 19:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wjames111 wjames111 wjames111 left review comments

At least 1 approving review is required to merge this pull request.

Assignees

@canac canac

Labels

Preview Environment Add this label to create an Amplify Preview

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@canac @wjames111