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

bug: update status badge when there's a new status #528

Open
TreyWW opened this issue Nov 8, 2024 · 2 comments · May be fixed by #544
Open

bug: update status badge when there's a new status #528

TreyWW opened this issue Nov 8, 2024 · 2 comments · May be fixed by #544
Labels

Comments

@TreyWW
Copy link
Owner

TreyWW commented Nov 8, 2024

What version of MyFinances are you currently using?

latest

What device type are you currently facing the issue on?

No response

Describe the bug

At the moment there's no update of the status (e.g. "paid") when you "mark as" until the page is reloaded. Would be ideal to either use HTMX with a custom trigger event, or just return the swap-oob on the response of mark (like our previous one)

image
image

@TreyWW TreyWW added the bug label Nov 8, 2024
@TreyWW TreyWW changed the title bug: update status badge when there's a new status via HTMX bug: update status badge when there's a new status Nov 8, 2024
@Angie-newbie
Copy link

Hello, I would like to pick up this issue and will begin working on it :)

@TreyWW
Copy link
Owner Author

TreyWW commented Jan 13, 2025

Hey @Angie-newbie, thanks for the interest! Sure thing, this feature should be conflict-less so thats okay. I've recently been doing some big changes to the majority of the files (around 400) so it's expected to have some merge issues - I'll try get this merged before my changes get merged to keep it flowing.

Do let me know if you need any help getting started, or implementing this. Thanks again for the contribution :)

@Angie-newbie Angie-newbie linked a pull request Jan 20, 2025 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants