Skip to content
This repository was archived by the owner on Apr 2, 2024. It is now read-only.

Update CLS Order URL path #79

Merged
philip-shippo merged 1 commit into from
Jun 6, 2022
Merged

Update CLS Order URL path #79

philip-shippo merged 1 commit into from
Jun 6, 2022

Conversation

@philip-shippo
Copy link
Contributor

@philip-shippo philip-shippo commented Jun 6, 2022 •
edited
Loading

closes #78

@epistemancer
Copy link

epistemancer commented Jun 6, 2022

I don't fully understand why this change is necessary -- I believe /orders/ and /v1/orders are the same endpoint -- but if nothing else it's keeping the paths consistent with other endpoints in the SDK.

@philip-shippo philip-shippo merged commit d8036fe into master Jun 6, 2022
@philip-shippo philip-shippo deleted the cls-url-path-update branch June 6, 2022 20:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

2 more reviewers

@tlaredo tlaredo tlaredo approved these changes

@epistemancer epistemancer epistemancer approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Order.create uses an invalid URL

4 participants

@philip-shippo @epistemancer @tlaredo