Skip to content
This repository has been archived by the owner on Aug 14, 2024. It is now read-only.

readme fix: Shipment.all – pass parameters as a hash #152

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Jonathan-Jacob
Copy link

Replaced this

Shipcloud::Shipment.all(
  carrier: 'ups',
  per_page: 25,
  page: 2
)

with this

Shipcloud::Shipment.all({
  carrier: 'ups',
  per_page: 25,
  page: 2
})

since otherwise you will get an ArgumentError

@Jonathan-Jacob
Copy link
Author

Failed checks should be irrelevant since this is only a PR for a README change

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant