Repository navigation
Land #75 and #76 on main - #81
Merged
Merged
Conversation
The Query page opened the ledger file a second time, read-only. Windows refuses that while the app's own connection holds the file, so every query failed with an IO error naming the app itself. It now clones the open connection and runs in a read-only transaction, which still refuses a write that gets past the keyword guard. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Bordered and filled buttons are one size, and black is kept for real commitments. Alert cards, Overview card links and the Accounts actions line up. Internal terms such as "Local ledger" are gone. SQL is highlighted with syntect, which also builds for the web demo. Charts gain axes and show a single day as a marker, and figures say whether they are net, before credits, or missing charges with no exchange rate. Alert and untagged-spend links open a query that answers them. Insights findings show their full ID, account and charges, and its cards wrap. The sidebar and Insights groups work from the keyboard and pages carry accessibility names. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The dedupe key carried the date, so a balance that stayed low added a card every day, with nothing to tell them apart. The key is now the account's, and the open card is rewritten with the latest figures. Dismissing quiets it for the day, a finished snooze reopens it once, and dated cards from before this change merge into the newest. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
It compared with the whole previous month, so the first days of any month showed falls of around 90%. It now stops the comparison after the same days, matching the Overview. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The page showed the whole period's findings, so a Claude model appeared under an OpenAI account, with shares of everyone's usage. The checks now take an account on both backends. Dismissing there covers that account only; a finding dismissed for the whole period stays dismissed. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
6 tasks
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.
#75 and #76 were merged into their stacked base branches (
feat/i18nandfix/ui-polish) rather thanmain, so their five commits never reachedmain. This PR lands them.It adds no new code: the head is the existing
feat/scoped-health-and-alerts, whose tree is identical tofix/ui-polishafter #76. A trial merge into currentmainhas no conflicts.Commits:
The manual checks listed on #75 and #76 still apply.
🤖 Generated with Claude Code