Skip to content

Remove states,countries,postcard.message#165

Merged
hwrdprkns merged 2 commits intomasterfrom
hwrd-deprecate
Nov 1, 2017
Merged

Remove states,countries,postcard.message#165
hwrdprkns merged 2 commits intomasterfrom
hwrd-deprecate

Conversation

@hwrdprkns
Copy link
Copy Markdown

@hwrdprkns hwrdprkns commented Oct 30, 2017

What: Remove the states api, countries api, and postcard.message field
Why: Newest API version deprecates those

@hwrdprkns hwrdprkns changed the title What: Remove the states api, countries api, and postcard.message field Why: Newest API version deprecates those Remove states,countries,postcard.message Oct 30, 2017
Copy link
Copy Markdown

@mgartner mgartner left a comment

Choose a reason for hiding this comment

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

Looks good. A couple more things to remove:

https://github.com/lob/lob-ruby/blob/hwrd-deprecate/lib/lob/client.rb#L61-L63

https://github.com/lob/lob-ruby/blob/hwrd-deprecate/lib/lob/client.rb#L41-L43

There's a few mentions of state and countries in README.md

@hwrdprkns
Copy link
Copy Markdown
Author

Correct - f'd that up when trying to split the changes into two commits locally

@coveralls
Copy link
Copy Markdown

coveralls commented Oct 30, 2017

Coverage Status

Coverage decreased (-0.04%) to 99.406% when pulling 08fc129 on hwrd-deprecate into c4544d8 on master.

Copy link
Copy Markdown

@mgartner mgartner left a comment

Choose a reason for hiding this comment

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

LGTM 💯

@coveralls
Copy link
Copy Markdown

coveralls commented Oct 30, 2017

Coverage Status

Coverage decreased (-0.04%) to 99.401% when pulling 7ec1f32 on hwrd-deprecate into c4544d8 on master.

Copy link
Copy Markdown

@vinay-pad vinay-pad left a comment

Choose a reason for hiding this comment

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

LGTM!

@vinay-pad vinay-pad assigned hwrdprkns and unassigned vinay-pad Oct 31, 2017
@hwrdprkns hwrdprkns merged commit 506dcd3 into master Nov 1, 2017
@mpiercy827 mpiercy827 deleted the hwrd-deprecate branch December 8, 2017 17:36
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.

4 participants