[MDS-5976] AMS Payment contact info #3146
Conversation
henryoforeh-dev
requested review from
simensma-fresh,
matbusby-fw,
taraepp,
isuru-aot and
asinn134
| }, []); | ||
|
|
||
| return ( | ||
| <div className="ant-form-vertical" style={{ paddingTop: 12 }}> |
There was a problem hiding this comment.
Shouldn't need the "ant-form-vertical" classname, the parent already has it.
| </Col> | ||
| <Col md={12} sm={24}> | ||
| <Field | ||
| name="payment_contact.address[0].post_code" |
There was a problem hiding this comment.
I'm wondering about the validation on these. This one looks like it wants the postalCodeWithCountry validation, and is the country not even required? I think it autosaves with Canada if sent through as null.
There was a problem hiding this comment.
Added validation. On the figma design, all the address fields are optional. I think it is also in line with what Ana asked for during our meeting with her. The main information they want are the ones that are mandatory.
Figma design screenshot.
https://www.figma.com/design/O7i6jpbKbju49HfL1OpozT/CORE-%26-Minespace-2021-2023?node-id=30995-90032&t=0utC7RRmtI5R9Cfj-0
|
|
|
|
Objective
MDS-5976
Why are you making this change? Provide a short explanation and/or screenshots