Skip to content

Show no after-tax figures without a tax model - #625

Open
clstaudt wants to merge 1 commit into
cash-flow-tax-ratefrom
no-tax-figures-without-tax-model
Open

clstaudt wants to merge 1 commit into
cash-flow-tax-ratefrom
no-tax-figures-without-tax-model

Conversation

@clstaudt

@clstaudt clstaudt commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

Without a tax model, income tax counted as 0. The Tax view's Safe to Spend and reserve rate, the Salary page and the cash flow showed figures with no tax in them; only the dashboard card said "—".

  • Core: a missing tax model raises NoTaxRules instead of returning zero tax. Figures up to the tax base stay. Income tax, expenses charged on taxed income and Safe to Spend are None, with the reason.
  • Tax view: "—" and the reason, e.g. "No tax model for Norway."
  • Salary: the reason instead of a salary.
  • Cash flow: inflows and expenses stay; one line says income tax isn't included, and why.

🤖 Generated with Claude Code

@clstaudt
clstaudt added this pull request to stack #620 October 18, 2026 20:23
Without a tax model for the operating country, the income tax estimate
came out as zero. The Tax view's Safe to Spend and reserve rate, the
Salary page and the cash flow's outflows were computed with that zero and
shown as real figures; only the dashboard card showed a dash.

A missing tax model is now reported instead of taken as zero. Figures up
to the tax base stay. Income tax, the expenses charged on taxed income,
Safe to Spend and the salary show a dash, with one line saying what is
missing, such as "No tax model for Norway." The cash flow keeps its
inflows and expenses and says that it leaves out the income tax.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@clstaudt
clstaudt force-pushed the no-tax-figures-without-tax-model branch from f661fce to d1c5a83 Compare October 20, 2026 13:48

This branch has not been deployed

No deployments
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.

1 participant