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

Test/codehike doc #787

Merged
merged 11 commits into from
Nov 8, 2023
Merged

Test/codehike doc #787

merged 11 commits into from
Nov 8, 2023

Conversation

carolinamenezes
Copy link
Collaborator

Types of changes

  • New content (guides, endpoints, app documentation)
  • Improvement (make a documentation even better)
  • Fix (fix a documentation error)
  • Spelling and grammar accuracy (self-explanatory)

Copy link
Contributor

github-actions bot commented Nov 1, 2023

Navigation Preview Link

No changes detected in the navigation.json file

Copy link
Contributor

github-actions bot commented Nov 1, 2023

Preview Links

Open this URL to set up the portal with this branch changes.

You can now access the edited pages with the following URLs:

Copy link
Collaborator

@julia-rabello julia-rabello left a comment

Choose a reason for hiding this comment

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

Alguns comentários mais gerais sobre formatação dos blocos do codehike:

  • O fato de não ter word-wrap nos code blocks gera a necessidade de scroll lateral para ver a URL completa da chamada nos blocos de cURL. Não sei se existe essa possibilidade, mas talvez seja bom ver se a gente consegue testar uma versão com word-wrap.
  • Se for possível customizar a altura / tamanho dos blocos (exemplo da Stripe), em casos como o da imagem abaixo, acho que o ideal seria o cURL ter uma altura menor para dar mais espaço para o bloco de baixo:
    image

docs/guides/Catalog/managing-skus.mdx Show resolved Hide resolved
docs/guides/Catalog/managing-skus.mdx Outdated Show resolved Hide resolved
docs/guides/Catalog/managing-skus.mdx Show resolved Hide resolved
docs/guides/Catalog/managing-skus.mdx Show resolved Hide resolved
docs/guides/Catalog/managing-skus.mdx Outdated Show resolved Hide resolved
docs/guides/Catalog/managing-skus.mdx Show resolved Hide resolved
docs/guides/Catalog/managing-skus.mdx Outdated Show resolved Hide resolved
docs/guides/Catalog/managing-skus.mdx Outdated Show resolved Hide resolved
docs/guides/Catalog/managing-skus.mdx Show resolved Hide resolved
docs/guides/Catalog/managing-skus.mdx Outdated Show resolved Hide resolved
@@ -6,8 +6,11 @@ createdAt: "2021-10-25T23:13:41.287Z"
updatedAt: "2022-10-27T18:56:50.737Z"
---
There are different ways of using VTEX APIs to handle shopping carts and checkout in order to place orders. For instance, you can use API requests to [create](https://developers.vtex.com/docs/api-reference/checkout-api#get-/api/checkout/pub/orderForm) and [manage](https://developers.vtex.com/docs/api-reference/checkout-api#post-/api/checkout/pub/orderForm/-orderFormId-/attachments/shippingData) shopping carts on the VTEX platform, so as to place an order from that information later, or directly place an order with a single request containing all cart data.

> ℹ️️ The main data structure used in VTEX Checkout is the `orderForm`. It contains every piece of information pertinent to a shopping cart, including logistics, payment, products and customer profile, for instance. Learn more in the [orderForm documentation.](https://developers.vtex.com/docs/guides/orderform-fields)

Copy link
Contributor

Choose a reason for hiding this comment

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

[markdownlint] reported by reviewdog 🐶
MD028/no-blanks-blockquote Blank line inside blockquote

@carolinamenezes carolinamenezes merged commit 1787754 into main Nov 8, 2023
3 of 4 checks passed
@carolinamenezes carolinamenezes deleted the test/codehike-doc branch November 8, 2023 16:19
@carolinamenezes carolinamenezes self-assigned this May 9, 2024
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