Skip to content

Conversation

@T-Thiry
Copy link

@T-Thiry T-Thiry commented Aug 31, 2025

Tavan Thiry added 30 commits August 15, 2025 11:26
:wq# the commit.
…endpoints of the API in server.js, add authmiddleware.js and jsonwebtoken
Tavan Thiry added 23 commits August 22, 2025 21:03
…s and DELETE /income, and fix bgColor warning
…oading states for balance and spendings cards
@JennieDalgren JennieDalgren self-assigned this Oct 9, 2025
Copy link
Contributor

@JennieDalgren JennieDalgren left a comment

Choose a reason for hiding this comment

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

Great job with this project, however it looks like your API calls are towards your local backend.
image

Please swap this out to your deployed backend so I can test your project in order to get going with the grading.

In general, I like your design and color theme. Very well polished and feels like a "complete" product design wise.

Now I just need to see that the functionality works too.
Let me know when it is fixed.

@T-Thiry
Copy link
Author

T-Thiry commented Oct 19, 2025

Thank you for your feedback @JennieDalgren

Absolutely, I’m working on it! The local version is working, but not the deployed one, as you pointed out.

@T-Thiry
Copy link
Author

T-Thiry commented Oct 26, 2025

@JennieDalgren I’ve made the necessary updates. You can now add both income and spending entries. The total values are displayed on the dashboard, and the balance is calculated as income minus spending. You’ve probably noticed that the variable expenses, fixed expenses, and monthly summary aren’t dynamic yet, only the results and the chart are. There’s still quite a bit to improve in terms of design consistency, accessibility, and overall functionality.

Note that it may take a few minutes for the page to load. It took me almost five minutes and several tries before everything appeared, but it does work once it loads.

Copy link
Contributor

@JennieDalgren JennieDalgren left a comment

Choose a reason for hiding this comment

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

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