Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add missing question option translations for PartYearProfitTaxCreditsFlow #2020

Conversation

floehopper
Copy link
Contributor

Expected user-visible changes

Visit this example URL.

Before the change this resulted in a 503 error page with the following message: "Sorry, we are experiencing technical difficulties".

After the change it should display the "Have you stopped trading?" question page correctly:

2015-10-19 at 10 39

## Technical details

This flow was relying on the fallback to the option key which was removed
in #2005. The following exception has just occurred in production 1:

I18n::MissingTranslationData: translation missing:
  en-GB.flow.part-year-profit-tax-credits.have_you_stopped_trading?.options.yes

This commit adds unit tests which would've picked up the problem and adds
the missing translations.

This flow was relying on the fallback to the option key which was removed
in #2005. The following exception has just occurred in production [1]:

    I18n::MissingTranslationData: translation missing:
      en-GB.flow.part-year-profit-tax-credits.have_you_stopped_trading?.options.yes

This commit adds unit tests which would've picked up the problem and adds
the missing translations.

[1]: https://errbit.publishing.service.gov.uk/apps/533c35ae0da1159384044f5f/problems/56233f4b6578634c6c971600
@floehopper floehopper force-pushed the add-missing-question-option-translations-for-part-year-profit-tax-credits branch from c936866 to 138808e Compare October 18, 2015 11:41
@tadast
Copy link
Contributor

tadast commented Oct 19, 2015

LGTM 👍

floehopper added a commit that referenced this pull request Oct 19, 2015
…anslations-for-part-year-profit-tax-credits

Add missing question option translations for PartYearProfitTaxCreditsFlow
@floehopper floehopper merged commit 5b1ec64 into master Oct 19, 2015
@floehopper floehopper deleted the add-missing-question-option-translations-for-part-year-profit-tax-credits branch October 19, 2015 09:20
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.

2 participants