Skip to content

Conversation

@suzuber
Copy link
Contributor

@suzuber suzuber commented May 21, 2025

No description provided.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 31e2612:

✅ Validation status: passed

File Status Preview URL Details
articles/sap/toc.yml ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@prmerger-automator
Copy link
Contributor

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 07cb685:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/sap/enterprise-insights/enterprise-insights-overview.md ⚠️Warning Details
articles/sap/toc.yml ✅Succeeded

articles/sap/enterprise-insights/enterprise-insights-overview.md

  • Line 6, Column 13: [Warning: ms-service-subservice-invalid - See documentation] Invalid value for 'ms.service': 'enterprise-insights'.
  • Line 27, Column 1: [Warning: file-not-found - See documentation] Invalid file link: 'ei-introduction.png'.
  • Line 31, Column 1: [Warning: file-not-found - See documentation] Invalid file link: 'ei-solution-overview.png'.
  • Line 45, Column 1: [Warning: file-not-found - See documentation] Invalid file link: 'ei-reference-architecture.png'.
  • Line 86, Column 3: [Warning: file-not-found - See documentation] Invalid file link: 'deployment-guide.md'.

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.

For any questions, please:

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 531fc2f:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/sap/enterprise-insights/enterprise-insights-overview.md ⚠️Warning Details
articles/sap/enterprise-insights/media/enterprise-insights-introduction.png ✅Succeeded
articles/sap/enterprise-insights/media/enterprise-insights-solution-overview.png ✅Succeeded
articles/sap/toc.yml ✅Succeeded

articles/sap/enterprise-insights/enterprise-insights-overview.md

  • Line 45, Column 1: [Warning: file-not-found - See documentation] Invalid file link: 'ei-reference-architecture.png'.
  • Line 86, Column 3: [Warning: file-not-found - See documentation] Invalid file link: 'deployment-guide.md'.

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.

For any questions, please:

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit e881f62:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/sap/enterprise-insights/enterprise-insights-overview.md ⚠️Warning Details
articles/sap/enterprise-insights/enterprise-insights-tutorial-prerequisites.md 💡Suggestion Details
articles/sap/enterprise-insights/media/enterprise-insights-introduction.png ✅Succeeded
articles/sap/enterprise-insights/media/enterprise-insights-solution-overview.png ✅Succeeded
articles/sap/toc.yml ✅Succeeded

articles/sap/enterprise-insights/enterprise-insights-overview.md

  • Line 45, Column 1: [Warning: file-not-found - See documentation] Invalid file link: 'ei-reference-architecture.png'.
  • Line 86, Column 3: [Warning: file-not-found - See documentation] Invalid file link: 'deployment-guide.md'.
  • Line 3, Column 14: [Suggestion: duplicate-descriptions - See documentation] Attribute 'description' with value 'Overview of Enterprise Insights for SAP data integration and analytics in Microsoft Fabric.' is duplicated in 'articles/sap/enterprise-insights/enterprise-insights-overview.md(3,14)', 'articles/sap/enterprise-insights/enterprise-insights-tutorial-prerequisites.md(3,14)'.

articles/sap/enterprise-insights/enterprise-insights-tutorial-prerequisites.md

  • Line 3, Column 14: [Suggestion: duplicate-descriptions - See documentation] Attribute 'description' with value 'Overview of Enterprise Insights for SAP data integration and analytics in Microsoft Fabric.' is duplicated in 'articles/sap/enterprise-insights/enterprise-insights-overview.md(3,14)', 'articles/sap/enterprise-insights/enterprise-insights-tutorial-prerequisites.md(3,14)'.

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.

For any questions, please:

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 358fa66:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/sap/enterprise-insights/enterprise-insights-overview.md ⚠️Warning Details
articles/sap/enterprise-insights/enterprise-insights-tutorial-prerequisites.md ✅Succeeded
articles/sap/enterprise-insights/media/enterprise-insights-introduction.png ✅Succeeded
articles/sap/enterprise-insights/media/enterprise-insights-solution-overview.png ✅Succeeded
articles/sap/toc.yml ✅Succeeded

articles/sap/enterprise-insights/enterprise-insights-overview.md

  • Line 45, Column 1: [Warning: file-not-found - See documentation] Invalid file link: 'ei-reference-architecture.png'.
  • Line 86, Column 3: [Warning: file-not-found - See documentation] Invalid file link: 'deployment-guide.md'.

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.

For any questions, please:

@v-dirichards
Copy link
Contributor

@suzuber Did you mean to make these changes in the private repo? This PR was opened in the public repo.

Microsoft authors and contributors need to make content contributions through the private repository so the content can be staged and validated by the current validation rules.

After you make your changes in the private repo, review the articles on staging, and fix any validation issues, you can add the sign-off comment to let PR reviewers know that the updates are ready to be merged.

Read about authors working in the public repo for more information.

If you need help getting set up to work in the private repo, see Get started contributing to the Microsoft Learn platform.

We'll close this PR. Thank you.

@jjaygbay1
Copy link
Contributor

@suzuber Did you mean to make these changes in the private repo? This PR was opened in the public repo.

Microsoft authors and contributors need to make content contributions through the private repository so the content can be staged and validated by the current validation rules.

After you make your changes in the private repo, review the articles on staging, and fix any validation issues, you can add the sign-off comment to let PR reviewers know that the updates are ready to be merged.

Read about authors working in the public repo for more information.

If you need help getting set up to work in the private repo, see Get started contributing to the Microsoft Learn platform.

We'll close this PR. Thank you.

@v-dirichards Suzi is working in the public repo outside the MSFT network. Is that a problem?

@v-dirichards
Copy link
Contributor

@jjaygbay1 - Oh I see. I saw they had editing perms in the private repo, so that's why I left that message.

This content will require automated and human checks available only in the private repository. Please move the commits to the private repository. If you need help moving the commits, contact the publicprs alias.

@suzuber
Copy link
Contributor Author

suzuber commented May 22, 2025

@v-dirichards I need to finish my commits before having Jacob move them into the private repo. Okay to reopen this PR?

@v-dirichards v-dirichards reopened this May 23, 2025
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 3fd4bcb:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/sap/enterprise-insights/enterprise-insights-overview.md ⚠️Warning Details
articles/sap/enterprise-insights/enterprise-insights-tutorial-prerequisites.md ✅Succeeded
articles/sap/enterprise-insights/media/enterprise-insights-introduction.png ✅Succeeded
articles/sap/enterprise-insights/media/enterprise-insights-solution-overview.png ✅Succeeded
articles/sap/toc.yml ✅Succeeded

articles/sap/enterprise-insights/enterprise-insights-overview.md

  • Line 45, Column 1: [Warning: file-not-found - See documentation] Invalid file link: 'ei-reference-architecture.png'.
  • Line 86, Column 3: [Warning: file-not-found - See documentation] Invalid file link: 'deployment-guide.md'.

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.

For any questions, please:

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 3fd4bcb:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/sap/enterprise-insights/enterprise-insights-overview.md ⚠️Warning Details
articles/sap/enterprise-insights/enterprise-insights-tutorial-prerequisites.md ✅Succeeded
articles/sap/enterprise-insights/media/enterprise-insights-introduction.png ✅Succeeded
articles/sap/enterprise-insights/media/enterprise-insights-solution-overview.png ✅Succeeded
articles/sap/toc.yml ✅Succeeded

articles/sap/enterprise-insights/enterprise-insights-overview.md

  • Line 45, Column 1: [Warning: file-not-found - See documentation] Invalid file link: 'ei-reference-architecture.png'.
  • Line 86, Column 3: [Warning: file-not-found - See documentation] Invalid file link: 'deployment-guide.md'.

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.

For any questions, please:

@v-regandowner v-regandowner added the aq-pr-triaged tracking label for the PR review team label May 23, 2025
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 73cda9e:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/sap/enterprise-insights/enterprise-insights-overview.md ⚠️Warning Details
articles/sap/enterprise-insights/enterprise-insights-tutorial-prerequisites.md ✅Succeeded
articles/sap/enterprise-insights/media/enterprise-insights-introduction.png ✅Succeeded
articles/sap/enterprise-insights/media/enterprise-insights-solution-overview.png ✅Succeeded
articles/sap/toc.yml ✅Succeeded

articles/sap/enterprise-insights/enterprise-insights-overview.md

  • Line 45, Column 1: [Warning: file-not-found - See documentation] Invalid file link: 'ei-reference-architecture.png'.
  • Line 86, Column 3: [Warning: file-not-found - See documentation] Invalid file link: 'deployment-guide.md'.

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.

For any questions, please:

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 96edb74:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/sap/enterprise-insights/enterprise-insights-overview.md ⚠️Warning Details
articles/sap/enterprise-insights/enterprise-insights-tutorial-prerequisites.md ✅Succeeded
articles/sap/enterprise-insights/media/enterprise-insights-introduction.png ✅Succeeded
articles/sap/enterprise-insights/media/enterprise-insights-solution-overview.png ✅Succeeded
articles/sap/toc.yml ✅Succeeded

articles/sap/enterprise-insights/enterprise-insights-overview.md

  • Line 45, Column 1: [Warning: file-not-found - See documentation] Invalid file link: 'ei-reference-architecture.png'.
  • Line 86, Column 3: [Warning: file-not-found - See documentation] Invalid file link: 'deployment-guide.md'.

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.

For any questions, please:

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit b606124:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/sap/enterprise-insights/enterprise-insights-overview.md ⚠️Warning Details
articles/sap/enterprise-insights/enterprise-insights-tutorial-prerequisites.md ✅Succeeded
articles/sap/enterprise-insights/media/enterprise-insights-introduction.png ✅Succeeded
articles/sap/enterprise-insights/media/enterprise-insights-reference-architecture.png ✅Succeeded
articles/sap/enterprise-insights/media/enterprise-insights-solution-overview.png ✅Succeeded
articles/sap/toc.yml ✅Succeeded

articles/sap/enterprise-insights/enterprise-insights-overview.md

  • Line 86, Column 3: [Warning: file-not-found - See documentation] Invalid file link: 'deployment-guide.md'.

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.

For any questions, please:

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 1bc099f:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/sap/enterprise-insights/enterprise-insights-overview.md ⚠️Warning Details
articles/sap/enterprise-insights/enterprise-insights-tutorial-prerequisites.md ✅Succeeded
articles/sap/enterprise-insights/media/enterprise-insights-introduction.png ✅Succeeded
articles/sap/enterprise-insights/media/enterprise-insights-reference-architecture.png ✅Succeeded
articles/sap/enterprise-insights/media/enterprise-insights-solution-overview.png ✅Succeeded
articles/sap/enterprise-insights/toc.yml ✅Succeeded
articles/sap/toc.yml ✅Succeeded

articles/sap/enterprise-insights/enterprise-insights-overview.md

  • Line 86, Column 3: [Warning: file-not-found - See documentation] Invalid file link: 'deployment-guide.md'.

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.

For any questions, please:

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 6b43621:

✅ Validation status: passed

File Status Preview URL Details
articles/sap/enterprise-insights/enterprise-insights-overview.md ✅Succeeded
articles/sap/enterprise-insights/enterprise-insights-tutorial-prerequisites.md ✅Succeeded
articles/sap/enterprise-insights/media/enterprise-insights-introduction.png ✅Succeeded
articles/sap/enterprise-insights/media/enterprise-insights-reference-architecture.png ✅Succeeded
articles/sap/enterprise-insights/media/enterprise-insights-solution-overview.png ✅Succeeded
articles/sap/enterprise-insights/toc.yml ✅Succeeded
articles/sap/toc.yml ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@prmerger-automator
Copy link
Contributor

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.
Image File This PR added or updated an image file(s).

@learn-build-service-prod
Copy link
Contributor

@prmerger-automator
Copy link
Contributor

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.
Image File This PR added or updated an image file(s).

@v-dirichards
Copy link
Contributor

@suzuber Are you still making changes to this draft, or is it ready for @ggailey777's input?

@suzuber
Copy link
Contributor Author

suzuber commented Jun 16, 2025

@v-dirichards yes, we are still making changes and awaiting publication date, tbd. Thank you for asking.

@ggailey777
Copy link
Contributor

@v-dirichards i don't think I'm the correct person to review this 😮

@v-dirichards
Copy link
Contributor

#review: @ritikesh-vali

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces a new Enterprise Insights section to the SAP documentation with comprehensive documentation for SAP data integration and analytics solutions in Microsoft Fabric.

  • Adds new Enterprise Insights documentation section with overview and prerequisites
  • Updates main SAP table of contents to include the new section
  • Creates structured documentation covering solution architecture, features, and implementation requirements

Reviewed Changes

Copilot reviewed 4 out of 7 changed files in this pull request and generated 2 comments.

File Description
articles/sap/toc.yml Adds Enterprise Insights section to main SAP navigation
articles/sap/enterprise-insights/toc.yml Creates new table of contents structure for Enterprise Insights documentation
articles/sap/enterprise-insights/enterprise-insights-overview.md Provides comprehensive overview of Enterprise Insights solution and architecture
articles/sap/enterprise-insights/enterprise-insights-tutorial-prerequisites.md Documents prerequisites and implementation requirements for Enterprise Insights

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment on lines +43 to +46
- [Replicate Runtime Hierarchy | SAP Help Portal](https://community.sap.com/)
- [Setting Report Relevancy for New Cost Center and Profit Center Hierarchies – SAP Help Portal](https://help.sap.com/docs/SAP_S4HANA_ON-PREMISE/5e23dc8fe9be4fd496f8ab556667ea05/0097f2ca29d549ec8641f5ccd3c8aebe.html)
- [How to synchronize Set Controlling Hierarchies in SAP Community](https://community.sap.com/)
- [Manage your hierarchies in FIORI 1/3 - SAP Community](https://community.sap.com/) No newline at end of file
Copy link

Copilot AI Sep 24, 2025

Choose a reason for hiding this comment

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

These three links all point to the generic SAP Community homepage (https://community.sap.com/) rather than specific documentation pages. The link text suggests they should point to specific articles about hierarchies and FIORI management.

Suggested change
- [Replicate Runtime Hierarchy | SAP Help Portal](https://community.sap.com/)
- [Setting Report Relevancy for New Cost Center and Profit Center Hierarchies – SAP Help Portal](https://help.sap.com/docs/SAP_S4HANA_ON-PREMISE/5e23dc8fe9be4fd496f8ab556667ea05/0097f2ca29d549ec8641f5ccd3c8aebe.html)
- [How to synchronize Set Controlling Hierarchies in SAP Community](https://community.sap.com/)
- [Manage your hierarchies in FIORI 1/3 - SAP Community](https://community.sap.com/)
- [Replicate Runtime Hierarchy | SAP Help Portal](https://help.sap.com/docs/SAP_S4HANA_ON-PREMISE/5e23dc8fe9be4fd496f8ab556667ea05/0097f2ca29d549ec8641f5ccd3c8aebe.html)
- [Setting Report Relevancy for New Cost Center and Profit Center Hierarchies – SAP Help Portal](https://help.sap.com/docs/SAP_S4HANA_ON-PREMISE/5e23dc8fe9be4fd496f8ab556667ea05/0097f2ca29d549ec8641f5ccd3c8aebe.html)
- [How to synchronize Set Controlling Hierarchies in SAP Community](https://blogs.sap.com/2021/03/10/how-to-synchronize-set-controlling-hierarchies-in-sap-s-4hana/)
- [Manage your hierarchies in FIORI 1/3 - SAP Community](https://blogs.sap.com/2022/02/14/manage-your-hierarchies-in-fiori-1-3/)

Copilot uses AI. Check for mistakes.
Comment on lines +43 to +46
- [Replicate Runtime Hierarchy | SAP Help Portal](https://community.sap.com/)
- [Setting Report Relevancy for New Cost Center and Profit Center Hierarchies – SAP Help Portal](https://help.sap.com/docs/SAP_S4HANA_ON-PREMISE/5e23dc8fe9be4fd496f8ab556667ea05/0097f2ca29d549ec8641f5ccd3c8aebe.html)
- [How to synchronize Set Controlling Hierarchies in SAP Community](https://community.sap.com/)
- [Manage your hierarchies in FIORI 1/3 - SAP Community](https://community.sap.com/) No newline at end of file
Copy link

Copilot AI Sep 24, 2025

Choose a reason for hiding this comment

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

These three links all point to the generic SAP Community homepage (https://community.sap.com/) rather than specific documentation pages. The link text suggests they should point to specific articles about hierarchies and FIORI management.

Suggested change
- [Replicate Runtime Hierarchy | SAP Help Portal](https://community.sap.com/)
- [Setting Report Relevancy for New Cost Center and Profit Center Hierarchies – SAP Help Portal](https://help.sap.com/docs/SAP_S4HANA_ON-PREMISE/5e23dc8fe9be4fd496f8ab556667ea05/0097f2ca29d549ec8641f5ccd3c8aebe.html)
- [How to synchronize Set Controlling Hierarchies in SAP Community](https://community.sap.com/)
- [Manage your hierarchies in FIORI 1/3 - SAP Community](https://community.sap.com/)
- [Replicate Runtime Hierarchy | SAP Help Portal](https://help.sap.com/docs/SAP_S4HANA_ON-PREMISE/5e23dc8fe9be4fd496f8ab556667ea05/0097f2ca29d549ec8641f5ccd3c8aebe.html)
- [Setting Report Relevancy for New Cost Center and Profit Center Hierarchies – SAP Help Portal](https://help.sap.com/docs/SAP_S4HANA_ON-PREMISE/5e23dc8fe9be4fd496f8ab556667ea05/0097f2ca29d549ec8641f5ccd3c8aebe.html)
- [How to synchronize Set Controlling Hierarchies in SAP Community](https://blogs.sap.com/2021/03/10/how-to-synchronize-set-controlling-hierarchies-in-sap-s-4hana/)
- [Manage your hierarchies in FIORI 1/3 - SAP Community](https://blogs.sap.com/2022/02/15/manage-your-hierarchies-in-fiori-1-3/)

Copilot uses AI. Check for mistakes.
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.

6 participants