8000 Fix/invoices refresh button function by XinSkyrim · Pull Request #563 · TreyWW/MyFinances · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix/invoices refresh button function #563

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

XinSkyrim
Copy link
@XinSkyrim XinSkyrim commented Apr 8, 2025

Description

Fix the refresh button function.

When I clicked the refresh button, I re-listened for the click event with js and sent the get request by ajax. In pyhton, I also modified the fetch function, which can identify carried parameters for dynamic filtering.

Attention : Be sure to merge Add a time range filter under invoices before merging this request. Because I need the front end to pass the correct date format before comparing the data in the database for filtering

Checklist

  • Ran the Black Formatter and
    djLint-er on any new code
    (checks
    will
    fail without)
  • Made any changes or additions to the documentation where required
  • Changes generate no new warnings/errors
  • New and existing unit tests pass locally with my
    changes

What type of PR is this?

  • 🐛 Bug Fix

Added/updated tests?

  • 🙅 no, because they aren't needed

Related PRs, Issues etc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0