-
Notifications
You must be signed in to change notification settings - Fork 179
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
Added PendingShipment request call to this repository #37
base: master
Are you sure you want to change the base?
Conversation
… time in the rate request
@sangeetamuthu / @ricaurte thanks for the PR, if you could add some tests for this would be awesome. Let me know if I can be of any assistance. Thanks!! |
end | ||
|
||
|
||
end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems like quite much extra space, IMHO :)
I think it would be a bit more readable if we'll put a line in between function definition and attr_accessor instead. What do you think?
it allows to generate label url that can be sent via email