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

feat(tax): TAX-1995 Add specs for new TaxZone check endpoint #12

Merged
merged 1 commit into from
Jan 18, 2024

Conversation

benpratt77
Copy link
Contributor

@benpratt77 benpratt77 commented Dec 28, 2023

DEVDOCS-5714

Add documentation in api-specs repo for our new ZoneCheck endpoint
https://bigcommercecloud.atlassian.net/browse/TAX-1995

What changed?

  • Added new spec file for the tax/zonecheck endpoint

Anything else?

https://github.com/bigcommerce/api-proxy-java/pull/2659#issuecomment-1859527805
Specs match the response:
Screenshot 2023-12-29 at 10 28 52 am

ping @bigcommerce/team-shipping

theromulans
theromulans previously approved these changes Jan 16, 2024
Copy link
Contributor

@theromulans theromulans left a comment

Choose a reason for hiding this comment

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

Thank you sir

@benpratt77 benpratt77 merged commit 3cf75cf into main Jan 18, 2024
4 checks passed
@benpratt77 benpratt77 deleted the TAX-1995 branch January 18, 2024 02:47
@bc-andreadao
Copy link
Collaborator

Work continued in PR 54

content:
application/json:
schema:
type: array
Copy link
Collaborator

Choose a reason for hiding this comment

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

Testing around, it seems like the request is an object (instead of an array), so I fixed the doc in PR 54

zone check

type: object
properties:
data:
type: array
Copy link
Collaborator

Choose a reason for hiding this comment

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

Same thing here, my testing showed data is an object, not an array, so fixed in PR 54

bc-tgomez pushed a commit that referenced this pull request Apr 26, 2024
# [DEVDOCS-5714]
Add documentation in api-specs repo for our new ZoneCheck endpoint
https://bigcommercecloud.atlassian.net/browse/TAX-1995

## What changed?
* Added new spec file for the `tax/zonecheck` endpoint

## Anything else?

bigcommerce/api-proxy-java#2659 (comment)
Specs match the response:
<img width="1309" alt="Screenshot 2023-12-29 at 10 28 52 am"
src="https://github.com/bigcommerce/docs/assets/40374950/c84912fd-bb52-45af-962c-30c2cdb49b82">

ping @bigcommerce/team-shipping 

[DEVDOCS-5714]:
https://bigcommercecloud.atlassian.net/browse/DEVDOCS-5714?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
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.

5 participants