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

Rename pay-leave-for-parents question pages #2018

Merged
merged 14 commits into from
Oct 19, 2015

Conversation

chrisroos
Copy link
Contributor

We're planning to migrate pay-leave-for-parents from Smartdown to Ruby in the near future.

We can use the convert-smartdown-flow-to-smart-answer-flow.rb to automatically convert some of this Smart Answer, but it won't work for some of the more complicated conditionals. We'll need to convert those by hand and I think that having a one-to-one mapping between question page name (i.e. filename) and question will make that process easier.

Expected changes

  • No user-facing changes expected.

@chrisroos chrisroos force-pushed the rename-pay-leave-for-parents-question-pages branch 2 times, most recently from 69fa766 to 92a5589 Compare October 15, 2015 17:44
@floehopper floehopper self-assigned this Oct 19, 2015
@floehopper
Copy link
Contributor

LGTM 👍

So that it matches the name of the question it contains.
So that it matches the name of the question it contains.
So that it matches the name of the question it contains.
So that it matches the name of the question it contains.
So that it matches the name of the question it contains.
So that it matches the name of the question it contains.
So that it matches the name of the question it contains.
So that it matches the name of the question it contains.
So that it matches the name of the question it contains.
So that it matches the name of the question it contains.
So that it matches the name of the question it contains.
So that it matches the name of the question it contains.
So that it matches the name of the question it contains.
So that it matches the name of the question it contains.
@chrisroos chrisroos force-pushed the rename-pay-leave-for-parents-question-pages branch from 92a5589 to 0c9121d Compare October 19, 2015 10:29
@chrisroos
Copy link
Contributor Author

I've rebased on master and force pushed in preparation for merging.

chrisroos added a commit that referenced this pull request Oct 19, 2015
…uestion-pages

Rename pay-leave-for-parents question pages
@chrisroos chrisroos merged commit ce334a1 into master Oct 19, 2015
@chrisroos chrisroos deleted the rename-pay-leave-for-parents-question-pages branch October 19, 2015 10:34
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