Skip to content
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

FFS-873: Expense Ledger Add Screen #5

Merged
merged 7 commits into from
Jun 8, 2024
Merged

Conversation

JosephGasiorekUSDS
Copy link
Collaborator

JIRA
Confluence requirements

NOTE: Use of the React USWDS DatePicker component has added a warning to the build. The team that owns the component is aware of it.

Screenshots:
Screen Shot 2024-06-07 at 10 32 08-fullpage

Screen Shot 2024-06-07 at 10 26 31-fullpage

@@ -7,7 +7,7 @@ import { useAppDispatch } from "@/lib/hooks"
import { addIncome, IncomeItem } from "@/lib/features/ledger/income/incomeSlice"
import { useRouter } from "next/navigation"
import { FieldErrors, SubmitHandler, useForm, Controller } from "react-hook-form"
import TextFieldWithValidation from "./TextFieldWithValidation"
import TextFieldWithValidation from "../../../components/TextFieldWithValidation"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As mentioned earlier, figure out a way to remove the .. pathing.

Also, why does the TextFieldWithValidation file start with a capitalized letter while other tsx files start with lowercase? Is there a rationale. Thinking of conventions here and being consistent for better readability.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@YvetteWhiteUSDS I updated the path's to use the @ shortcut to the root of the app. Regarding the component casing, I capitalized it to follow the pattern of the React USWDS library.

@JosephGasiorekUSDS JosephGasiorekUSDS merged commit 6c36bee into main Jun 8, 2024
1 check passed
@JosephGasiorekUSDS JosephGasiorekUSDS deleted the jgasiorek-ffs-873 branch June 28, 2024 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants