Skip to content

372 dev upload cash starting amount personal benefits increase and personal salary increase#378

Open
camila-carrillo wants to merge 3 commits intomainfrom
372-dev---upload-cash-starting-amount-personal-benefits-increase-and-personal-salary-increase
Open

372 dev upload cash starting amount personal benefits increase and personal salary increase#378
camila-carrillo wants to merge 3 commits intomainfrom
372-dev---upload-cash-starting-amount-personal-benefits-increase-and-personal-salary-increase

Conversation

@camila-carrillo
Copy link
Copy Markdown
Contributor

ℹ️ Issue

Closes #372

📝 Description

Made cash starting amount, personnel benefits increase, and personnel salary increase functional. The fields are now editable, connected to store, and saved to backend on logout so that changes will persist between sessions.

Briefly list the changes made to the code:

  1. Added CashflowSettings type to middle layer
  2. Added cashflowSettings field to store, actions, and mutators
  3. Added fetchCashflowSettings to fetch settings from the backend endpoint
  4. Added saveCashflowSettings to save all three settings to backend on logout
  5. Updated CashPosition and CashAnnualSettings to be editable and connected to store
  6. Updated NavBar logout handler to save cashflow settings before clearing the store
  7. Updated default-values controller from patch to put to fix CORS blocking issue

✔️ Verification

What steps did you take to verify your changes work? These should be clear enough for someone to be able to clone the branch and follow the steps themselves.

Tested locally. Confirmed values change after logout in DynamoDB table.

Provide screenshots of any new components, styling changes, or pages.

Test Changes

If your new feature required some test to be changed or added to fit the new functionality or changes please document these changes here.

🏕️ (Optional) Future Work / Notes

Did you notice anything ugly during the course of this ticket? Any bugs, design challenges, or unexpected behavior? Write it down so we can clean it up in a future ticket!

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.

DEV - Upload cash starting amount, personal benefits increase, and personal salary increase

1 participant