Skip to content

fix: say VAT instead of moms in English copy - #474

Open
MartinPTielemans wants to merge 1 commit into
mainfrom
fix/qui-286-english-vat-copy
Open

MartinPTielemans wants to merge 1 commit into
mainfrom
fix/qui-286-english-vat-copy

Conversation

@MartinPTielemans

Copy link
Copy Markdown
Contributor

Fixes QUI-286

The English Settings, dashboard and onboarding copy still used the Danish word "moms". It now says VAT.

  • Settings: "Moms period" → "VAT return period", help text "report moms to Skat" → "file your VAT return with Skat", toast "Moms period saved." → "VAT return period saved."
  • Dashboard deadline: "Moms for {period} is due …" → "VAT return for {period} is due …" (3 strings)
  • VAT status hint: "You invoice without moms." → "You invoice without VAT."

Kept: the five "(momsloven)" citations in PDF and e-invoice exemption text, since they name the Danish VAT Act.

Danish copy, identifiers, contract fields and stored data are unchanged. A new guard test fails if any English catalog string contains "moms" outside a "(momsloven)" citation.

@linear-code

linear-code Bot commented Oct 24, 2026

Copy link
Copy Markdown

QUI-286

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ⚠️ Failed 2026-10-24T14:02:30.232303Z
We couldn’t complete this review. Comment @codex review to try again.
87d7183 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

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