-
Notifications
You must be signed in to change notification settings - Fork 19
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
Adding the new CapitalFinancingPromotion component #209
Adding the new CapitalFinancingPromotion component #209
Conversation
Switch the old CapitalOverview embedded component to the new CapitalFinancingPromotion embedded component. This new embedded component will display details of a financing offer for the account if there is one.
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.
Besides my suggestion, and the formatting issue, this looks great!
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.
I agree with @streeter-stripe 's suggestion, but other than that, looks great!
Fix lint errors and add switch statement to render the Capital offer promotion.
Updating formatting with Prettier
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.
looks good to me!
Switch the old CapitalOverview embedded component to the new CapitalFinancingPromotion embedded component. This new embedded component will display details of a financing offer for the account if there is one.