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

Update checksum data for redundancy smart answers #2004

Merged
merged 2 commits into from
Oct 14, 2015

Conversation

chrisroos
Copy link
Contributor

I made a change to redundancy_pay.rb in another branch and was surprised not to have to update the checksum data. I realised that it was because the checksum data didn't include the redundancy_pay shared logic.

This branch rectifies that.

Expected user-visible changes

  • None.

@floehopper
Copy link
Contributor

LGTM 👍

I noticed that the checksum data didn't include the redundancy_pay.rb shared
logic.

Updated using:

    $ rails r script/generate-checksums-for-smart-answer.rb \
    calculate-employee-redundancy-pay \
    lib/smart_answer_flows/shared_logic/redundancy_pay.rb
I noticed that the checksum data didn't include the redundancy_pay.rb shared
logic.

Updated using:

    $ rails r script/generate-checksums-for-smart-answer.rb \
    calculate-your-redundancy-pay \
    lib/smart_answer_flows/shared_logic/redundancy_pay.rb
@chrisroos chrisroos force-pushed the update-checksum-data-for-redundancy-smart-answers branch from e22aede to f4d40c7 Compare October 14, 2015 09:54
@chrisroos
Copy link
Contributor Author

I've rebased on master and force pushed ready to merge.

chrisroos added a commit that referenced this pull request Oct 14, 2015
…dancy-smart-answers

Update checksum data for redundancy smart answers
@chrisroos chrisroos merged commit 8c26d9f into master Oct 14, 2015
@chrisroos chrisroos deleted the update-checksum-data-for-redundancy-smart-answers branch October 14, 2015 10:10
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