flipdot/drinks-touch
22
0
Fork 0
Code Issues 3 Pull requests 3 Projects Releases Packages Wiki Activity Actions

Allow users to delete their transaction history #35

Open
soerface wants to merge 8 commits from delete-tx into main
pull from: delete-tx
merge into: flipdot:main
Conversation 1 Commits 8 Files changed 8 +271 -22
soerface commented 2026-06-07 21:16:36 +00:00
Owner
Copy link
No description provided.
Allow users to delete their transaction history
Some checks failed
CI / CI optimization (pull_request) Has been cancelled
Details
CI / Release (semantic, dry) (pull_request) Has been cancelled
Details
CI / pre-commit (pull_request) Has been cancelled
Details
CI / Build (pull_request) Has been cancelled
Details
CI / Release (semantic) (pull_request) Has been cancelled
Details
cab6a1689f
Simplify logic
Some checks failed
CI / CI optimization (pull_request) Has been cancelled
Details
CI / Release (semantic, dry) (pull_request) Has been cancelled
Details
CI / pre-commit (pull_request) Has been cancelled
Details
CI / Build (pull_request) Has been cancelled
Details
CI / Release (semantic) (pull_request) Has been cancelled
Details
fe144d6a17
Enable CSRF protection
Some checks failed
CI / CI optimization (pull_request) Has been cancelled
Details
CI / Release (semantic, dry) (pull_request) Has been cancelled
Details
CI / pre-commit (pull_request) Has been cancelled
Details
CI / Build (pull_request) Has been cancelled
Details
CI / Release (semantic) (pull_request) Has been cancelled
Details
3787fc9c95
Redirect if method is GET instead of showing error
Some checks failed
CI / CI optimization (pull_request) Has been cancelled
Details
CI / Release (semantic, dry) (pull_request) Has been cancelled
Details
CI / pre-commit (pull_request) Has been cancelled
Details
CI / Build (pull_request) Has been cancelled
Details
CI / Release (semantic) (pull_request) Has been cancelled
Details
dba60e32c3
Move button disabled style to style.css
Some checks failed
CI / CI optimization (pull_request) Has been cancelled
Details
CI / Release (semantic, dry) (pull_request) Has been cancelled
Details
CI / pre-commit (pull_request) Has been cancelled
Details
CI / Build (pull_request) Has been cancelled
Details
CI / Release (semantic) (pull_request) Has been cancelled
Details
516cf26501
Delete tx set to 1970-01-01 for privacy reasons, and to easier squash history in the future (will add a feature to delete all transactions older than X days)
Some checks failed
CI / CI optimization (pull_request) Has been cancelled
Details
CI / Release (semantic, dry) (pull_request) Has been cancelled
Details
CI / pre-commit (pull_request) Has been cancelled
Details
CI / Build (pull_request) Has been cancelled
Details
CI / Release (semantic) (pull_request) Has been cancelled
Details
8da1df6554
For plotting purposes, assume the 1970 transactions to be from the otherwise oldest transaction of that user
Some checks failed
CI / CI optimization (pull_request) Has been cancelled
Details
CI / Release (semantic, dry) (pull_request) Has been cancelled
Details
CI / pre-commit (pull_request) Has been cancelled
Details
CI / Build (pull_request) Has been cancelled
Details
CI / Release (semantic) (pull_request) Has been cancelled
Details
faa95c95c8
Merge branch 'main' into delete-tx
Some checks failed
CI / CI optimization (pull_request) Has been cancelled
Details
CI / Release (semantic, dry) (pull_request) Has been cancelled
Details
CI / pre-commit (pull_request) Has been cancelled
Details
CI / Build (pull_request) Has been cancelled
Details
CI / Release (semantic) (pull_request) Has been cancelled
Details
28b34d6bcb
malled approved these changes 2026-06-11 19:15:18 +00:00
drinks_touch/database/balance.py
@ -50,0 +60,4 @@
total_amount = sum(t.amount for t in transactions)
deleted_count = len(transactions)
for t in transactions:
malled commented 2026-06-11 19:12:49 +00:00
Owner
Copy link

Instead of iterating over all transactions, the delete query with the previous filter can be used.

Instead of iterating over all transactions, the delete query with the previous filter can be used.
Some checks failed
CI / CI optimization (pull_request) Has been cancelled
Details
CI / Release (semantic, dry) (pull_request) Has been cancelled
Details
CI / pre-commit (pull_request) Has been cancelled
Details
CI / Build (pull_request) Has been cancelled
Details
CI / Release (semantic) (pull_request) Has been cancelled
Details
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin delete-tx:delete-tx
git switch delete-tx

Merge

Merge the changes and update on Forgejo.
git switch main
git merge --no-ff delete-tx
git switch delete-tx
git rebase main
git switch main
git merge --ff-only delete-tx
git switch delete-tx
git rebase main
git switch main
git merge --no-ff delete-tx
git switch main
git merge --squash delete-tx
git switch main
git merge --ff-only delete-tx
git switch main
git merge delete-tx
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
malled
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
malled
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
flipdot/drinks-touch!35
No description provided.