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

Marriage abroad: Two outcome path for luxembourg #3174

Merged
merged 4 commits into from
Sep 4, 2017

Conversation

ikennaokpala
Copy link
Contributor

@ikennaokpala ikennaokpala commented Aug 2, 2017

Trello card

Description

This pull request reduces Luxembourg to 2 outcome paths:

  • 1 for same sex
  • 1 for opposite sex

Content design team have provided the content which have also been included.

Factcheck

Preview link
GOVUK

Expected changes

  • Reduction of outcomes from 18 to 2
  • Enable 2 outcome flow
  • Removal of all redundant erb templates and artefacts
  • Update content for Luxembourg

Before

screen shot 2017-08-02 at 15 34 54

After

screen shot 2017-08-02 at 15 34 13

@ikennaokpala ikennaokpala added needs content review Waiting for a content designer to approve Ready for code review labels Aug 2, 2017
@chao-xian chao-xian temporarily deployed to smart-answers-preview-pr-3174 August 2, 2017 14:31 Inactive
@ikennaokpala ikennaokpala force-pushed the marriage-abroad/2-outcome-path-for-luxembourg branch from 04ef373 to ff9c50f Compare August 2, 2017 15:15
@chao-xian chao-xian temporarily deployed to smart-answers-preview-pr-3174 August 2, 2017 15:15 Inactive
@ikennaokpala ikennaokpala force-pushed the marriage-abroad/2-outcome-path-for-luxembourg branch from ff9c50f to 45b979a Compare August 3, 2017 16:32
@chao-xian chao-xian temporarily deployed to smart-answers-preview-pr-3174 August 3, 2017 16:32 Inactive
@ikennaokpala ikennaokpala added Waiting on factcheck and removed needs content review Waiting for a content designer to approve labels Aug 3, 2017
This commit switches Luxembourg from an 18 to 2 outcome country.

This include:

- Moving Luxembourg to the 2 outcome country list
- Adding opposite and same sex partials (this will be updated later)
- Updating the responses and expected response yaml file
- Updating the integration tests

The aforementioned changes have made the erb templates under ceremony_country
third_country and uk redundant and affected the regression test artefacts.

These will be removed and updated in comming commits respectively.
Sequel to the commit before this, this commit introduces the two
outcome paths:

- /marriage-abroad/y/luxembourg/opposite_sex
- /marriage-abroad/y/luxembourg/same_sex

It removes the following redundant erb templates:

- lib/smart_answer_flows/marriage-abroad/outcomes/countries/luxembourg/ceremony_country/partner_british/_opposite_sex.erb
- lib/smart_answer_flows/marriage-abroad/outcomes/countries/luxembourg/ceremony_country/partner_british/_same_sex.erb
- lib/smart_answer_flows/marriage-abroad/outcomes/countries/luxembourg/ceremony_country/partner_local/_opposite_sex.erb
- lib/smart_answer_flows/marriage-abroad/outcomes/countries/luxembourg/ceremony_country/partner_local/_same_sex.erb
- lib/smart_answer_flows/marriage-abroad/outcomes/countries/luxembourg/ceremony_country/partner_other/_opposite_sex.erb
- lib/smart_answer_flows/marriage-abroad/outcomes/countries/luxembourg/ceremony_country/partner_other/_same_sex.erb
- lib/smart_answer_flows/marriage-abroad/outcomes/countries/luxembourg/third_country/partner_british/_opposite_sex.erb
- lib/smart_answer_flows/marriage-abroad/outcomes/countries/luxembourg/third_country/partner_british/_same_sex.erb
- lib/smart_answer_flows/marriage-abroad/outcomes/countries/luxembourg/third_country/partner_local/_opposite_sex.erb
- lib/smart_answer_flows/marriage-abroad/outcomes/countries/luxembourg/third_country/partner_local/_same_sex.erb
- lib/smart_answer_flows/marriage-abroad/outcomes/countries/luxembourg/third_country/partner_other/_opposite_sex.erb
- lib/smart_answer_flows/marriage-abroad/outcomes/countries/luxembourg/third_country/partner_other/_same_sex.erb
- lib/smart_answer_flows/marriage-abroad/outcomes/countries/luxembourg/uk/partner_british/_opposite_sex.erb
- lib/smart_answer_flows/marriage-abroad/outcomes/countries/luxembourg/uk/partner_british/_same_sex.erb
- lib/smart_answer_flows/marriage-abroad/outcomes/countries/luxembourg/uk/partner_local/_opposite_sex.erb
- lib/smart_answer_flows/marriage-abroad/outcomes/countries/luxembourg/uk/partner_local/_same_sex.erb
- lib/smart_answer_flows/marriage-abroad/outcomes/countries/luxembourg/uk/partner_other/_opposite_sex.erb
- lib/smart_answer_flows/marriage-abroad/outcomes/countries/luxembourg/uk/partner_other/_same_sex.erb

And also no longer supports the following 18 outcome paths:

- /marriage-abroad/y/luxembourg/ceremony_country/partner_british/opposite_sex
- /marriage-abroad/y/luxembourg/ceremony_country/partner_british/same_sex
- /marriage-abroad/y/luxembourg/ceremony_country/partner_local/opposite_sex
- /marriage-abroad/y/luxembourg/ceremony_country/partner_local/same_sex
- /marriage-abroad/y/luxembourg/ceremony_country/partner_other/opposite_sex
- /marriage-abroad/y/luxembourg/ceremony_country/partner_other/same_sex
- /marriage-abroad/y/luxembourg/third_country/partner_british/opposite_sex
- /marriage-abroad/y/luxembourg/third_country/partner_british/same_sex
- /marriage-abroad/y/luxembourg/third_country/partner_local/opposite_sex
- /marriage-abroad/y/luxembourg/third_country/partner_local/same_sex
- /marriage-abroad/y/luxembourg/third_country/partner_other/opposite_sex
- /marriage-abroad/y/luxembourg/third_country/partner_other/same_sex
- /marriage-abroad/y/luxembourg/uk/partner_british/opposite_sex
- /marriage-abroad/y/luxembourg/uk/partner_british/same_sex
- /marriage-abroad/y/luxembourg/uk/partner_local/opposite_sex
- /marriage-abroad/y/luxembourg/uk/partner_local/same_sex
- /marriage-abroad/y/luxembourg/uk/partner_other/opposite_sex
- /marriage-abroad/y/luxembourg/uk/partner_other/same_sex
This commit updates the opposite and same sex outcomes for Luxembourg

The update has been carry out based on instructions received from
the content design team.

The outcome paths affected by these changes are:

- /marriage-abroad/y/luxembourg/opposite_sex
- /marriage-abroad/y/luxembourg/same_sex
@ikennaokpala ikennaokpala force-pushed the marriage-abroad/2-outcome-path-for-luxembourg branch from 45b979a to cfc4ff7 Compare September 4, 2017 09:58
@chao-xian chao-xian temporarily deployed to smart-answers-preview-pr-3174 September 4, 2017 09:58 Inactive
This content update changes the country from France to Luxembourg and
enumerates the possible card companies that are accepted for payments.

Checksum and regression test artefacts have been updated.
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.

None yet

2 participants