feat: add data plots and add create expense #11

nicolasauler · 2023-12-25T23:39:11Z

Added Expense over Time plot to index page.
Also, added creating expense functionality to frontend. Now user can manually add an expense, which is located in a modal.

More things done:

Getting expenses and plots on page load

⚠️ I forked plotly, because plotly has askama as dependency, but without with-axum feature flag, that creates a clash with the dependencies of finnish. So I forked it and added askama_axum as a dependency there.
This it definitely not optimal, but such is life ⚠️

TODO:

Close modal upon clicking outside of it
Maybe whenever an expense is created, the table and plot could be changed to whatever month the added expense was attributed to. Idk, though

Also began expense creation in htmx.

However, a few bugs came up: - When we create an expense, search and edit become not functional and break everything. - When we create an expense, it should trigger a get in the month issued in the request to show now all expenses. - Also, if we click outside the modal, it doesn't work, modal isn't closed.

TODO: include created expense date in table GET. This way, we would be able to apply logic to maybe change to the month of the created date. Idk

Needed because of dependency clashes with upstream plotly: later make a PR.

This reverts commit 97a0e83.

This way I can fix the dependency clashes with the deps of plotly. This is not supposed to be a permanent fix, though.

confused when it comes to decimal separator.

nicolasauler added 5 commits December 24, 2023 20:00

Added plotly plot of price in period.

7ef2e5a

Also began expense creation in htmx.

Added htmx refresh expense table trigger after creating an expense.

e25fbc6

TODO: include created expense date in table GET. This way, we would be able to apply logic to maybe change to the month of the created date. Idk

Preparing for deploy, sanitized codebase.

63f4c29

Added triggers to make page more responsive.

865d1e0

nicolasauler self-assigned this Dec 25, 2023

nicolasauler added 4 commits December 25, 2023 20:58

Added plotly.

97a0e83

Needed because of dependency clashes with upstream plotly: later make a PR.

Revert "Added plotly."

c7065e6

This reverts commit 97a0e83.

Created a fork of plotly haha.

6fce51d

This way I can fix the dependency clashes with the deps of plotly. This is not supposed to be a permanent fix, though.

Added a placeholder for price in creating expense, so user is not

2fd60a9

confused when it comes to decimal separator.

nicolasauler merged commit 672c21b into main Dec 27, 2023
2 checks passed

nicolasauler deleted the feat-data-plots branch December 27, 2023 19:48

nicolasauler added this to the Expenses overview with manual operation milestone Dec 27, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: add data plots and add create expense #11

feat: add data plots and add create expense #11

nicolasauler commented Dec 25, 2023 •

edited

Loading

feat: add data plots and add create expense #11

feat: add data plots and add create expense #11

Conversation

nicolasauler commented Dec 25, 2023 • edited Loading

nicolasauler commented Dec 25, 2023 •

edited

Loading