Skip to content

Conversation

@v-chgri
Copy link
Contributor

@v-chgri v-chgri commented Sep 30, 2025

Pull request guidance

Thank you for submitting your contribution to our support content! Our team works closely with subject matter experts in CSS and PMs in the product group to review all content requests to ensure technical accuracy and the best customer experience. This process can sometimes take one or more days, so we greatly appreciate your patience.

We also need your help in order to process your request as soon as possible:

  • We won't act on your pull request (PR) until you type "#sign-off" in a new comment in your pull request (PR) to indicate that your changes are complete.

  • After you sign off in your PR, the article will be tech reviewed by the PM or SME if it has more than minor changes. Once the article is approved, it will undergo a final editing pass before being merged.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit a45c987:

❌ Validation status: errors

Please follow instructions here which may help to resolve issue.

File Status Preview URL Details
support/dynamics-365/commerce/toc.yml ❌Error Details
support/dynamics-365/commerce/payments/adyen-connector-eft-terminal-id.md 💡Suggestion Details

support/dynamics-365/commerce/toc.yml

  • Line 67, Column 6: [Error: yaml-syntax-error - See documentation] While parsing a block collection, did not find expected '-' indicator.

support/dynamics-365/commerce/payments/adyen-connector-eft-terminal-id.md

  • Line 0, Column 0: [Suggestion: filename-incomplete - See documentation] The filename contains an incomplete word, misspelled word, acronym, or abbreviation that is disallowed.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 0bbb1d9:

💡 Validation status: suggestions

File Status Preview URL Details
support/dynamics-365/commerce/payments/adyen-connector-eft-terminal-id.md 💡Suggestion Details
support/dynamics-365/commerce/payments/adyen-connector-invoicing-sales-orders-fail-stale-authorization.md ✅Succeeded
support/dynamics-365/commerce/payments/adyen-connector-store-commerce-app-error.md ✅Succeeded
support/dynamics-365/commerce/toc.yml ✅Succeeded
support/dynamics-365/commerce/welcome-commerce.yml ✅Succeeded

support/dynamics-365/commerce/payments/adyen-connector-eft-terminal-id.md

  • Line 0, Column 0: [Suggestion: filename-incomplete - See documentation] The filename contains an incomplete word, misspelled word, acronym, or abbreviation that is disallowed.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@prmerger-automator
Copy link

PRMerger Results

Issue Description
Added File(s) This PR contains added files. New files require human review.
Yaml File(s) This PR includes changes to .yml file(s) owned by another author.
File Change Percent This PR contains file(s) with more than 30% file change.

@learn-build-service-prod
Copy link
Contributor

@prmerger-automator
Copy link

PRMerger Results

Issue Description
Added File(s) This PR contains added files. New files require human review.
Yaml File(s) This PR includes changes to .yml file(s) owned by another author.
File Change Percent This PR contains file(s) with more than 30% file change.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 3b4e295:

⚠️ Validation status: warnings

File Status Preview URL Details
support/dynamics-365/commerce/data-synchronization/commerce-offline-implementation.md ⚠️Warning Details
support/dynamics-365/commerce/payments/adyen-connector-invoicing-sales-orders-fail-stale-authorization.md ✅Succeeded
support/dynamics-365/commerce/payments/adyen-connector-store-commerce-app-error.md ✅Succeeded
support/dynamics-365/commerce/payments/adyen-connector-terminal.md ✅Succeeded
support/dynamics-365/commerce/toc.yml ✅Succeeded
support/dynamics-365/commerce/welcome-commerce.yml ✅Succeeded

support/dynamics-365/commerce/data-synchronization/commerce-offline-implementation.md

  • Line 9, Column 1: [Warning: h1-empty - See documentation] H1 is required. Add content to your top-level heading.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 9bf10f2:

⚠️ Validation status: warnings

File Status Preview URL Details
support/dynamics-365/commerce/data-synchronization/commerce-offline-implementation.md ⚠️Warning Details
support/dynamics-365/commerce/payments/adyen-connector-invoicing-sales-orders-fail-stale-authorization.md ✅Succeeded
support/dynamics-365/commerce/payments/adyen-connector-store-commerce-app-error.md ✅Succeeded
support/dynamics-365/commerce/payments/adyen-connector-terminal.md ✅Succeeded
support/dynamics-365/commerce/toc.yml ✅Succeeded
support/dynamics-365/commerce/welcome-commerce.yml ✅Succeeded

support/dynamics-365/commerce/data-synchronization/commerce-offline-implementation.md

  • Line 9, Column 1: [Warning: h1-empty - See documentation] H1 is required. Add content to your top-level heading.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@learn-build-service-prod
Copy link
Contributor

@prmerger-automator
Copy link

PRMerger Results

Issue Description
Added File(s) This PR contains added files. New files require human review.
Yaml File(s) This PR includes changes to .yml file(s) owned by another author.
File Change Percent This PR contains file(s) with more than 30% file change.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 878e1ce:

⚠️ Validation status: warnings

File Status Preview URL Details
support/dynamics-365/commerce/data-synchronization/commerce-data-exchange.md ⚠️Warning Details
support/dynamics-365/commerce/data-synchronization/commerce-offline-implementation.md ✅Succeeded
support/dynamics-365/commerce/payments/adyen-connector-invoicing-sales-orders-fail-stale-authorization.md ✅Succeeded
support/dynamics-365/commerce/payments/adyen-connector-store-commerce-app-error.md ✅Succeeded
support/dynamics-365/commerce/payments/adyen-connector-terminal.md ✅Succeeded
support/dynamics-365/commerce/toc.yml ✅Succeeded
support/dynamics-365/commerce/welcome-commerce.yml ✅Succeeded

support/dynamics-365/commerce/data-synchronization/commerce-data-exchange.md

  • Line 9, Column 1: [Warning: h1-empty - See documentation] H1 is required. Add content to your top-level heading.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit f6d7584:

⚠️ Validation status: warnings

File Status Preview URL Details
support/dynamics-365/commerce/data-synchronization/commerce-data-exchange.md ⚠️Warning Details
support/dynamics-365/commerce/data-synchronization/commerce-offline-implementation.md ✅Succeeded
support/dynamics-365/commerce/payments/adyen-connector-invoicing-sales-orders-fail-stale-authorization.md ✅Succeeded
support/dynamics-365/commerce/payments/adyen-connector-store-commerce-app-error.md ✅Succeeded
support/dynamics-365/commerce/payments/adyen-connector-terminal.md ✅Succeeded
support/dynamics-365/commerce/toc.yml ✅Succeeded
support/dynamics-365/commerce/welcome-commerce.yml ✅Succeeded

support/dynamics-365/commerce/data-synchronization/commerce-data-exchange.md

  • Line 9, Column 1: [Warning: h1-empty - See documentation] H1 is required. Add content to your top-level heading.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@prmerger-automator
Copy link

PRMerger Results

Issue Description
Added File(s) This PR contains added files. New files require human review.
Yaml File(s) This PR includes changes to .yml file(s) owned by another author.
Commits Count This PR contains more than 15 commits.
File Change Percent This PR contains file(s) with more than 30% file change.

@prmerger-automator
Copy link

PRMerger Results

Issue Description
Added File(s) This PR contains added files. New files require human review.
Yaml File(s) This PR includes changes to .yml file(s) owned by another author.
Commits Count This PR contains more than 15 commits.
File Change Percent This PR contains file(s) with more than 30% file change.

@prmerger-automator
Copy link

PRMerger Results

Issue Description
Added File(s) This PR contains added files. New files require human review.
Yaml File(s) This PR includes changes to .yml file(s) owned by another author.
Commits Count This PR contains more than 15 commits.
File Change Percent This PR contains file(s) with more than 30% file change.

@v-chgri
Copy link
Contributor Author

v-chgri commented Oct 15, 2025

#sign-off

@prmerger-automator
Copy link

Invalid command: '#sign-off'. Only Microsoft employees can use this command.

@rmca14
Copy link
Contributor

rmca14 commented Oct 15, 2025

@johnmichalak
Copy link

#sign-off

@prmerger-automator
Copy link

Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @v-chgri, @bstorie

@prmerger-automator
Copy link

PRMerger Results

Issue Description
Added File(s) This PR contains added files. New files require human review.
Yaml File(s) This PR includes changes to .yml file(s) owned by another author.
Commits Count This PR contains more than 15 commits.
File Change Percent This PR contains file(s) with more than 30% file change.

@prmerger-automator
Copy link

PRMerger Results

Issue Description
Added File(s) This PR contains added files. New files require human review.
Yaml File(s) This PR includes changes to .yml file(s) owned by another author.
Commits Count This PR contains more than 15 commits.
File Change Percent This PR contains file(s) with more than 30% file change.

@prmerger-automator
Copy link

PRMerger Results

Issue Description
Added File(s) This PR contains added files. New files require human review.
Yaml File(s) This PR includes changes to .yml file(s) owned by another author.
Commits Count This PR contains more than 15 commits.
File Change Percent This PR contains file(s) with more than 30% file change.

@johnmichalak
Copy link

#sign-off

@haywoodsloan
Copy link
Contributor

@v-chgri, I have copied this PR to the private repo for further discussion and edits: https://github.com/MicrosoftDocs/SupportArticles-docs-pr/pull/9955

cc: @aartig13 @johnmichalak

@v-chgri
Copy link
Contributor Author

v-chgri commented Oct 17, 2025

FYI I can't view the PR in the private repo (I'm off the corpnet for 6 more weeks). So I won't be able to access any feedback or discussions there. You can email me with questions & updates at [email protected].

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.

4 participants