Skip to content

Show no tax estimate for a year without its tax rules - #626

Merged
clstaudt merged 1 commit into
no-tax-figures-without-tax-modelfrom
no-tax-estimate-without-year-data
Oct 23, 2026
Merged

clstaudt merged 1 commit into
no-tax-figures-without-tax-modelfrom
no-tax-estimate-without-year-data

Conversation

@clstaudt

@clstaudt clstaudt commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

A year without its own tax rules was estimated with the nearest year's rules (#622 only labelled that). Now such a year has no estimate: "No tax rules for Germany in 2023." A country's currency no longer depends on the year, so conversions still work.

From 1 January of a year whose rules are missing, every tax estimate waits for them, and so do the tests that use the current year.

馃 Generated with Claude Code

@clstaudt
clstaudt added this pull request to stack #620 October 18, 2026 20:23
@clstaudt
clstaudt force-pushed the no-tax-estimate-without-year-data branch 3 times, most recently from 5613295 to 0a15c49 Compare October 23, 2026 21:41
Tax rules are kept in one file per year. For a year without its own
file, before the first or after the last, the engine took the nearest
year's rules, so the tax view estimated that year's income tax with
rules that were never in force for it and showed it like any other
estimate.

A year without its own rules now has none: the tax view, the salary and
the cash flow show no tax for it and say so, for example "No tax rules
for Germany in 2023." A country's currency no longer depends on the
year, so amounts are still converted into it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@clstaudt
clstaudt force-pushed the no-tax-estimate-without-year-data branch from 0a15c49 to 5cad195 Compare October 23, 2026 21:48
@clstaudt
clstaudt added this pull request to the merge queue Oct 23, 2026
Merged via the queue into main with commit 09d235d Oct 23, 2026
4 checks passed
@clstaudt
clstaudt deleted the no-tax-estimate-without-year-data branch October 23, 2026 22:07
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