-
Notifications
You must be signed in to change notification settings - Fork 2
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.
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
feat(fee-breakdown): Parse and display discrete list of reductions and exemptions #4040
feat(fee-breakdown): Parse and display discrete list of reductions and exemptions #4040
Conversation
5569c90
to
bd6ad07
Compare
a6d46c3
to
4a34dfc
Compare
4a34dfc
to
84338ad
Compare
Removed vultr server and associated DNS entries |
<FeeBreakdownSection /> | ||
<GovPayMetadataSection/> | ||
<InviteToPaySection/> | ||
<FeeBreakdownSection/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: understand this is just a linting change here, but going forward I think we'll want to re-order the Pay modal to better match the info-heirarchy of the public display? Eg should GovPayMetdataSection
be last because it's "internal"/on-submit only and FeeBreakdown
be before InviteToPay
because a) it'll always apply and b) it's visually higher on the page than the "Invite someone else to pay" button ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great observation, will pick up this change when I come back to the styling of this component shortly 👍
/** | ||
* Convert a Passport value to an actual boolean | ||
*/ | ||
const toBoolean = (val: ["true" | "false"]) => val[0] === "true"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's make a proper lib of passport helpers in planx-core or similar one of these days 💫
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Totally!
What does this PR do?
@descriptions
fields)