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

Tweaked function to update invoice status #544

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Angie-newbie
Copy link

@Angie-newbie Angie-newbie commented Jan 20, 2025

Description

At the moment there's no update of the status (e.g. "paid") when you "mark as" until the page is reloaded. (#528)

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

@Angie-newbie Angie-newbie marked this pull request as ready for review January 20, 2025 04:23
@TreyWW
Copy link
Owner

TreyWW commented Jan 20, 2025

Hey @Angie-newbie,

Thanks for the PR, great work. Interesting approach, we normally do all of this on the backend and return it using HTMX in the standard response.

It should be fine for now though! I'll get the typo test fixed shortly if I get the time.

Thanks a lot :)

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.

bug: update status badge when there's a new status
2 participants