Skip to content

Conversation

elnaz
Copy link
Contributor

@elnaz elnaz commented Nov 6, 2015

No description provided.

@elnaz elnaz force-pushed the update-for-new-version branch 2 times, most recently from ff027ed to 39187f5 Compare November 6, 2015 18:02
@elnaz elnaz changed the title [WIP] feat(upgrade): update all tests and examples to work with 2015-11-06 feat(upgrade): update all tests and examples to work with 2015-11-06 Nov 9, 2015
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think you use example_address elsewhere so you can delete it's creation

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good catch, I ended up reusing it as the check.to_address since the data param was using example_address.name for dynamic html.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@robinjoseph08
Copy link
Contributor

LGTM

@elnaz elnaz force-pushed the update-for-new-version branch from 39187f5 to 0b441d1 Compare November 9, 2015 22:29
@mgartner
Copy link
Contributor

mgartner commented Nov 9, 2015

LGTM

elnaz added a commit that referenced this pull request Nov 9, 2015
feat(upgrade): update all tests and examples to work with 2015-11-06
@elnaz elnaz merged commit 175ac13 into master Nov 9, 2015
@elnaz elnaz deleted the update-for-new-version branch November 9, 2015 23:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants