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

Updated API endpoints #73

Merged

Conversation

runmad
Copy link
Contributor

@runmad runmad commented Dec 9, 2015

What is this?

This PR primarily switches /anywhere/checkouts to the new /api/checkouts endpoint. However, I thought it was a good chance to switch over to using NSURLComponents to create all of the request URLs with, which consisted of some work to use the paths and query items that comes with it.

  • /anywhere/checkouts -> /api/checkouts
  • Use of NSURLComponents to create request URLs
  • methods that take a url have been switched to NSURL for better consistency
  • updated tests for collections with better assertions for the various components in a URL as well as query comparisons

@bgulanowski please review 👀 and 🎩

…en made using NSURLComponents. Tests have been updated to better match testing of the scheme, host, path and query items from which a NSURLComponents URL is created.
@bgulanowski
Copy link
Contributor

Looks fine. 🎩

runmad added a commit that referenced this pull request Dec 9, 2015
…-add-url-components

Updated API endpoints
@runmad runmad merged commit a697e62 into develop Dec 9, 2015
@runmad runmad deleted the chore/runmad.update-checkouts-api-add-url-components branch December 9, 2015 19:03
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.

2 participants