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

add services attribute to carriers call #5

Merged
merged 1 commit into from
Oct 26, 2015

Conversation

andrecedik
Copy link
Contributor

This PR adds handling of services to the carriers call

@@ -33,22 +43,54 @@
stub_carriers_request
allow(Shipcloud::Carrier).to receive(:new)

carriers = Shipcloud::Carrier.all
_carriers = Shipcloud::Carrier.all
Copy link
Contributor

Choose a reason for hiding this comment

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

Seems we don't need the assignment? I would suggest to remove it.

@sfroehler sfroehler self-assigned this Oct 26, 2015
@sfroehler
Copy link
Contributor

only one (minor) suggestions
ready to merge

@sfroehler sfroehler assigned andrecedik and unassigned sfroehler Oct 26, 2015
@andrecedik andrecedik force-pushed the ac-add-services-attribute-to-carriers-call branch from 1680fb5 to 3ca307e Compare October 26, 2015 15:18
andrecedik added a commit that referenced this pull request Oct 26, 2015
…riers-call

add services attribute to carriers call
@andrecedik andrecedik merged commit f0be7ee into master Oct 26, 2015
@andrecedik andrecedik deleted the ac-add-services-attribute-to-carriers-call branch October 26, 2015 15:33
@andrecedik andrecedik restored the ac-add-services-attribute-to-carriers-call branch October 26, 2015 15:33
@andrecedik andrecedik deleted the ac-add-services-attribute-to-carriers-call branch October 26, 2015 15:33
@andrecedik andrecedik restored the ac-add-services-attribute-to-carriers-call branch October 26, 2015 15:33
sfroehler added a commit that referenced this pull request Jan 21, 2016
- This CHANGELOG file (Following "[Keep a CHANGELOG](http://keepachangelog.com/)")
- Create, find, update and index operations for address resource. (#4)
- Services attribute to carriers call (#5)
- Index operation for shipment resources with optional filter parameters. (#6)
- Create, find and index operations for webhook resource. (#7)
- Added the following ruby versions to travis-ci test runs:
  - 2.1.7
  - 2.2.4
  - 2.3.0
- Add ShipmentQuotes class returning the price for a certain shipment (#9)

- Dropped support for ruby 1.9.x in order to use the new language features of ruby 2.x.
  The official support of ruby 1.9.3 already ended on February 23, 2015
  (https://www.ruby-lang.org/en/news/2014/01/10/ruby-1-9-3-will-end-on-2015/)
- Removed the following ruby versions from travis-ci test runs:
  - 2.1.5
  - 2.2.1
  - 2.2.2
  - 2.2.3

- Start following [SemVer](http://semver.org) properly.
- The link to the developer documenation for the Shipment ressource still pointed at Apiary;
  it now correctly points to the shipcloud Developer Portal's API section on Shipments.
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.

None yet

2 participants