Skip to content

MPDX-9334 - Save both average utility costs when one is modified#1652

Open
zweatshirt wants to merge 4 commits intomainfrom
MPDX-9334
Open

MPDX-9334 - Save both average utility costs when one is modified#1652
zweatshirt wants to merge 4 commits intomainfrom
MPDX-9334

Conversation

@zweatshirt
Copy link
Contributor

@zweatshirt zweatshirt commented Mar 17, 2026

Description

  • Modified the AutosaveCustomTextField to accept an additionalSaveFields prop which accepts an array of values to save.

Jira ticket

Testing

  • Go to /housingAllowance
  • Create a request or continue one
  • Check that the Average monthly utility costs line items both update when save occurs in the event of rentOrOwn being set to Own.

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

@zweatshirt zweatshirt self-assigned this Mar 17, 2026
@zweatshirt zweatshirt added the Preview Environment Add this label to create an Amplify Preview label Mar 17, 2026
@github-actions
Copy link
Contributor

Preview branch generated at https://MPDX-9334.d3dytjb8adxkk5.amplifyapp.com

@zweatshirt zweatshirt requested a review from wjames111 March 17, 2026 22:15
@github-actions
Copy link
Contributor

Bundle sizes [mpdx-react]

Compared against f82890e

No significant changes found

…nceRequestAttributesInput for attributes in saveValue function
@zweatshirt zweatshirt requested review from dr-bizz March 18, 2026 14:35
Copy link
Contributor

@dr-bizz dr-bizz left a comment

Choose a reason for hiding this comment

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

Is there a way we could make these the same fields? I guess if we did that, we would have to update the other when one is updated, and build similar functionality you have here.

Also, you need to make sure when you update the second utility field, that it

Image

also updates the Total Monthly Cost of Providing a Home and other totals.

Copy link
Contributor

@dr-bizz dr-bizz left a comment

Choose a reason for hiding this comment

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

I requested changes, but it says I didn't, so I'm requesting the same changes again.

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.

2 participants