8994 autogen notow counsel to test 1770308188#9716
Conversation
…ormation section and modal
…nsel-to-test-1770308188
There was a problem hiding this comment.
Pull request overview
This PR implements proper rendering of the petitioner's name and additionalName fields in the Notice of Withdrawal form and Edit Contact Information modal. The changes ensure that when a petitioner has an additional name (e.g., a representative, guardian, or trustee), it is displayed consistently with the "c/o" prefix.
Changes:
- Added display of additionalName with "c/o" prefix in NoticeOfWithdrawalForm and EditContactInformationModal
- Fixed a CSS class typo (
tw-block→tw:block) for international addresses - Added test coverage for the additionalName feature with Cypress integration test
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| web-client/src/views/FileDocument/NoticeOfWithdrawalForm.tsx | Added conditionally rendered additionalName display with "c/o" prefix; fixed CSS class typo for international country field |
| web-client/src/views/EditPetitionerInformationInternal.tsx | Added data-testid attribute to additionalName input for better test targeting; improved label formatting |
| web-client/src/views/CaseDetail/EditContactInformationModal.tsx | Replaced incorrect form.name with form.contact.name; added conditionally rendered additionalName display with "c/o" prefix |
| cypress/local-only/tests/integration/caseDetail/docketRecord/autoGeneratedDocuments/notice-of-withdrawal/contact-information.cy.ts | Added integration test that verifies additionalName is properly saved and displayed in the form |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <span className="tw:block tw:mb-4 tw:mt-4">{form.contact.name}</span> | ||
| {form.contact.additionalName && ( | ||
| <span className="tw:block tw:mb-4 tw:mt-4"> |
There was a problem hiding this comment.
The name and additionalName spans both have tw:mb-4 tw:mt-4 classes, which creates excessive vertical spacing (32px) between them when both are present. Consider removing the top and bottom margins from these elements to achieve more compact spacing that's consistent with the NoticeOfWithdrawalForm display (lines 178-183), where the name and additionalName are shown as consecutive block elements without additional margins.
| <span className="tw:block tw:mb-4 tw:mt-4">{form.contact.name}</span> | |
| {form.contact.additionalName && ( | |
| <span className="tw:block tw:mb-4 tw:mt-4"> | |
| <span className="tw:block">{form.contact.name}</span> | |
| {form.contact.additionalName && ( | |
| <span className="tw:block"> |
…court/ef-cms into 8994-autogen-notow-counsel-to-test-1770308188
Uh oh!
There was an error while loading. Please reload this page.