Repository navigation
fix(receipts): a way back from a receipt in the app, and included amounts are no lines - #184
Merged
Merged
Conversation
In the app on the home screen "Bonnetje openen" in Verwerken, and a receipt's thumbnail on an expense or in the tray, opened the file in a new tab that took over the screen with no way back, and what was typed in Verwerken was gone. There it now opens in a dialog with "Sluiten", as "Voorbeeld-pdf" does: a PDF in a frame, a photo as itself. In a browser tab, the desktop app and without JS nothing changes. A check makes every new tab inside Huismus carry an answer for the app. Playbook: n.v.t. — bugfix in Huismus zelf, geen werkwijze Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01W6o7JHCnFJVV7ub63VfxGH
A webshop's invoice lists a levy or recycling fee under an article as
"Incl. ... 2,80", already in the article's price. Read as lines of their
own, the lines came to more than the total, and Huismus rightly left the
receipt in the tray ("de bedragen tellen niet op tot het totaal"). The
prompt now says such amounts are never a line. Huismus's own check of
the lines against the total stays as it is.
Playbook: n.v.t. — bugfix in Huismus zelf, geen werkwijze
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01W6o7JHCnFJVV7ub63VfxGH
codebyarne
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Twee bugfixes rond bonnetjes, allebei door de eigenaar gevonden op de Pi. Elke commit gaat over één ding.
1. Een bonnetje in de app heeft een weg terug
In de app op het beginscherm van een telefoon opende "Bonnetje openen" bij Verwerken de pdf over het hele scherm, zonder weg terug. Wat je al had ingevuld, was dan weg. Hetzelfde gold voor een bonnetje bij een uitgave en bij Bonnetjes zonder uitgave.
tests/core/test_pwa.py): elke link of knop die een nieuw tabblad binnen Huismus opent, moet het antwoord van app.js voor de app hebben.2. Bedragen met "Incl." zijn geen eigen regel
De factuur van een webwinkel bleef liggen met "Niet gelezen: de bedragen tellen niet op tot het totaal". Onder een artikel stonden een heffing en een recyclingbijdrage met "Incl.", die al in de prijs zaten. Vermoedelijk telde het model die als losse regels mee. De vraag aan het model zegt nu dat zulke bedragen nooit een eigen regel zijn. Huismus' eigen check, dat de regels samen het totaal geven, blijft zoals hij was.
Elk platform: punt 1 verandert alleen de app op het beginscherm (Docker, telefoon); punt 2 werkt overal hetzelfde.
./scripts/verify.sh: groen.🤖 Generated with Claude Code
https://claude.ai/code/session_01W6o7JHCnFJVV7ub63VfxGH