-
Notifications
You must be signed in to change notification settings - Fork 1
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
All Expenses #375
All Expenses #375
Conversation
@@ -9,7 +9,7 @@ | |||
<div class="grid"> | |||
<div th:replace="~{fragments/goBack :: goBackLink}"></div> | |||
<main id="content" role="main" class="form-card spacing-above-35"> | |||
<p class="employer-name" th:text="${fieldData.get('employerName')}"></p> | |||
<p class="grey-text" th:text="${fieldData.get('employerName')}"></p> |
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.
fyi @lkemperman-cfa - changed the class to be more universal than employer name.
expensesHome: | ||
nextScreens: | ||
- name: expensesRent | ||
condition: ExpensesIncludeRent |
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.
@sree-cfa @lkemperman-cfa - I don't love this approach. Is there a better way?
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.
Yeah, I dont think there's a better way atm. The only thing I can think of is extracting some of the duplicate code into a super conditional class so that each condition just has to pass the corresponding value
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.
Ex.
return super.run(submission, "Rent")
No description provided.