Skip to content

Conversation

@eshanrnh
Copy link
Contributor

@eshanrnh eshanrnh commented Jun 18, 2024

Description

  • Added a folder for zero payment provider with Overview and Configuration articles.
  • Added zero heading in the summary file.

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)

When should the content be published?

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.

I've added a couple of comments.
Mainly structure related 💪

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 for taking a look at the suggestions.

The PR looks good to me 💪

@eshanrnh eshanrnh added the review/developer Use this label if an internal developer review is required label Jun 26, 2024
Copy link
Contributor

@mattbrailsford mattbrailsford left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@eshanrnh eshanrnh merged commit 4cf0376 into main Jul 2, 2024
@eshanrnh eshanrnh deleted the Zero-payment-provider branch July 5, 2024 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category/umbraco-commerce review/developer Use this label if an internal developer review is required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants