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

Fixing and improving Orders API #1345

Open
wants to merge 64 commits into
base: master
Choose a base branch
from

Conversation

Isabella-Veloso
Copy link
Contributor

Types of changes

  • New content (endpoints, descriptions or fields from scratch)
  • Improvement (make an endpoint's title or description even better)
  • Spelling and grammar accuracy (self-explanatory)

Changelog

Do not forget to update your changes to our Developer Portal's changelog. Did you create a release note?

  • Yes, I already created a release note about this change.
  • No, but I am going to.

Copy link

Thanks for your contribution. The .json file will be checked now with Spectral.

},
"orderGroup": {
"type": "string",
"description": "Order's group ID."
"description": "Order group ID, a code part of the order ID that groups all orders related to the same purchase. For example, in order ID `v71021570str-02`), the order group ID is `v71021570str`."
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"description": "Order group ID, a code part of the order ID that groups all orders related to the same purchase. For example, in order ID `v71021570str-02`), the order group ID is `v71021570str`."
"description": "Order group ID, a segment of the order ID that groups all orders related to the same purchase. For example, in the order ID `v71021570str-02`), the order group ID is `v71021570str`."

},
"hostname": {
"type": "string",
"description": "Account host name, which is the [domain](https://help.vtex.com/en/tutorial/configuring-the-store-domain--tutorials_2450#store-address-structure) registered in **Account Settings**."
"description": "Main [account name](https://help.vtex.com/en/tutorial/what-is-an-account-name--i0mIGLcg3QyEy8OCicEoC)."
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this really only the account name (e.g. apiexamples) or the full hostname (e.g. www.apiexamples.com)?

Asking because of the name of the parameter and the previous description including this doc https://help.vtex.com/en/tutorial/configuring-the-store-domain--tutorials_2450#store-address-structure

},
"roundingError": {
"type": "integer",
"description": "Rounding error total amount, if it applies. For example, in orders with a discount over non-integer multiplier items, the rounding price is performed per item, not after the sum of all items. That can cause a difference in the total discount amount, which is informed in this field."
"description": "Rounding error total value in cents, when applicable. For example, in orders with a discount over non-integer multiplier items, the rounding price is applied per item, not after the sum of all items, what can cause a difference in the total discount value."
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"description": "Rounding error total value in cents, when applicable. For example, in orders with a discount over non-integer multiplier items, the rounding price is applied per item, not after the sum of all items, what can cause a difference in the total discount value."
"description": "Rounding error total value in cents, when applicable. For example, in orders with discounts on non-integer multiplier items, rounding is applied per item, not after the sum of all items, which can result in a difference in the total discount value."

},
"end": {
"type": "string",
"description": "Subscriptions' ending date with the format `yyyy-mm-ddThh:mm:ss`.",
"description": "Subscription ending date in [ISO 8601 time zone offset format](https://learn.microsoft.com/en-us/rest/api/storageservices/formatting-datetime-values), as in `YYYY-MM-DDThh:mm:ssZ`.",
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"description": "Subscription ending date in [ISO 8601 time zone offset format](https://learn.microsoft.com/en-us/rest/api/storageservices/formatting-datetime-values), as in `YYYY-MM-DDThh:mm:ssZ`.",
"description": "Subscription end date in [ISO 8601 time zone offset format](https://learn.microsoft.com/en-us/rest/api/storageservices/formatting-datetime-values), as in `YYYY-MM-DDThh:mm:ssZ`.",

"type": "object",
"description": "Store's configuration data, stored in [Account settings](https://help.vtex.com/en/tutorial/account-details-page--2vhUVOKfCaswqLguT2F9xq).",
"description": "Customizable fields created by the store for the shopping cart. This field is useful for storing data not included in other fields. For example, a message for a gift or a name to be printed in a shirt.",
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this related to Catalog attachments / custom product attributes? (asking beacuse of the examples)

If so, I suggest linking to related docs.

Comment on lines +438 to +442
"description": "Customizable apps created by the store.",
"nullable": true,
"items": {
"type": "string",
"description": "Customizable apps details.",
Copy link
Contributor

Choose a reason for hiding this comment

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

What does this mean? Is there an example for this somewhere?

@@ -3270,7 +2935,7 @@
},
"orderGroup": {
"type": "string",
"description": "Order's group ID, a code part of the order ID that groupes all orders related to the same purchase. For example, in order ID `v71021570str-02`), the order group ID is `v71021570str`."
"description": "Order group ID, a code part of the order ID that groups all orders related to the same purchase. For example, in order ID `v71021570str-02`), the order group ID is `v71021570str`."
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"description": "Order group ID, a code part of the order ID that groups all orders related to the same purchase. For example, in order ID `v71021570str-02`), the order group ID is `v71021570str`."
"description": "Order group ID, a segment of the order ID that groups all orders related to the same purchase. For example, in order ID `v71021570str-02`), the order group ID is `v71021570str`."

@@ -13292,7 +12941,7 @@
},
"changesData": {
"type": "array",
"description": "Order changes details.",
"description": "Order changes information.",
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"description": "Order changes information.",
"description": "Information about changes in the order.",

@@ -20503,7 +20143,7 @@
"description": "Quantity of items been replaced."
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"description": "Quantity of items been replaced."
"description": "Quantity of items replaced."

@@ -20503,7 +20143,7 @@
"description": "Quantity of items been replaced."
},
"price": {
"type": "string",
"type": "integer",
"nullable": true,
"description": "Shipping price for the item been replaced. It does not account for the whole order's shipping price and is presented in cents."
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"description": "Shipping price for the item been replaced. It does not account for the whole order's shipping price and is presented in cents."
"description": "Shipping price for the replaced item, in cents. It does not account for the whole order's shipping price."

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