Skip to content

Conversation

@kegrimes
Copy link
Contributor

@kegrimes kegrimes commented Jan 16, 2026

Description

submittedDate, boardApprovedDate, approvedDate, and lastApprovedDate were duplicate fields after the addition of the new timestamps. These were removed from the backend, so this PR aims to update where they are used.

Testing

  • Go to /reports/housingAllowance
  • Click through the request form
  • Check that approved date is all good (only field seen on UI)

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

@kegrimes kegrimes added the Preview Environment Add this label to create an Amplify Preview label Jan 16, 2026
@github-actions
Copy link
Contributor

Preview branch generated at https://update-duplicate-mha-fields.d3dytjb8adxkk5.amplifyapp.com

@github-actions
Copy link
Contributor

Bundle sizes [mpdx-react]

Compared against c793717

No significant changes found

@kegrimes kegrimes requested a review from canac January 16, 2026 16:18
@kegrimes
Copy link
Contributor Author

@canac If this PR is ready to go, is #3126 good to go on staging?

Copy link
Contributor

@canac canac left a comment

Choose a reason for hiding this comment

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

Yeah, you can merge the API PR to staging!

@kegrimes kegrimes force-pushed the update-duplicate-mha-fields branch from 1f6642c to 29d8041 Compare January 16, 2026 16:38
@kegrimes kegrimes merged commit a191ce6 into main Jan 16, 2026
24 checks passed
@kegrimes kegrimes deleted the update-duplicate-mha-fields branch January 16, 2026 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Preview Environment Add this label to create an Amplify Preview

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants