Skip to content

Conversation

@mr-shellychauhan
Copy link
Contributor

Tutorial for Umbraco 13 Commerce to update cart

Description

Tutorial to update Commerce cart
What did you add/update/change?

Type of suggestion

  • Typo/grammar fix
  • Updated outdated content
  • New content
  • Updates related to a new version
  • Other

Product & version (if relevant)

Deadline (if relevant)

ASAP
When should the content be published?
ASAP

Tutorial for Umbraco 13 Commerce to update cart
@sofietoft sofietoft added review/developer Use this label if an internal developer review is required category/umbraco-commerce labels Oct 7, 2024
@sofietoft
Copy link
Contributor

Thanks for the PR @MrEssCee !

We'll get this reviewed as soon as possible.

Copy link
Contributor

@umbracotrd umbracotrd left a comment

Choose a reason for hiding this comment

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

fix the syntax error in the code

Copy link
Contributor

@sofietoft sofietoft left a comment

Choose a reason for hiding this comment

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

Thanks a bunch for the PR @MrEssCee ! 💪

I've taken the liberty to fix some grammar and style guide issues.

Also, could you look into adding the code snippets to code blocks, using the following docs: https://docs.gitbook.com/content-editor/blocks/code-block#representation-in-markdown
Don't mind about the line numbers and the wrap, but adding the file name would be great 🙌

@sofietoft
Copy link
Contributor

Also realized that the file name here is a bit wrong.
It needs to have the .md file extension to work with our docs 🙌

@sofietoft
Copy link
Contributor

Could you also add the article to the SUMMARY file?
Go up in the file structure and file the closest SUMMARY.md file - that will be for this version of the Commerce docs.
Add a link to this article, using the title of the article as the link title 💪

If it's not added to the SUMMARY file, it will not be part of the published docs.
Happy to do this as well - just let me know.

@sofietoft sofietoft merged commit 40307d1 into umbraco:main Oct 8, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants