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

CTA UX Version 2 description #10103

Draft
wants to merge 20 commits into
base: main
Choose a base branch
from
Draft

Conversation

ShreyasSar26
Copy link
Contributor

@ShreyasSar26 ShreyasSar26 commented Feb 5, 2025

Category

  • Content fix
  • New article

Related issues

What's in this Pull Request?

This PR documents the new changes introduced in SharePoint Admin Center to support management of SharePoint Embedded containers.

Copy link
Contributor

Learn Build status updates of commit 6050b77:

💡 Validation status: suggestions

File Status Preview URL Details
docs/embedded/concepts/admin-exp/consuming-tenant-admin/ctaUX.md 💡Suggestion View Details
docs/embedded/images/addowners-1.png ✅Succeeded View
docs/embedded/images/addowners-2.png ✅Succeeded View
docs/embedded/images/Filter.png ✅Succeeded View
docs/embedded/images/Reassignuser.png ✅Succeeded View
docs/embedded/images/Removeuser.png ✅Succeeded View
docs/embedded/images/Sensitivitylabel.png ✅Succeeded View
docs/embedded/images/Sorting.png ✅Succeeded View

docs/embedded/concepts/admin-exp/consuming-tenant-admin/ctaUX.md

  • Line 177, Column 1: [Suggestion: duplicate-alt-text - See documentation] Alt text 'SPE Container owner' is duplicated. Within an article, alt text must be unique.
  • Line 179, Column 1: [Suggestion: duplicate-alt-text - See documentation] Alt text 'SPE Container owner' is duplicated. Within an article, alt text must be unique.

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:

@andrewconnell andrewconnell self-assigned this Feb 10, 2025
@andrewconnell andrewconnell added the pr:in-review Actively reviewing pull request label Feb 10, 2025
@andrewconnell
Copy link
Collaborator

andrewconnell commented Feb 10, 2025

Thank you for the PR, but it is incomplete. You must complete the instructions in the PR template found in your OP to help us better understand what & why this PR is for.

Please refer to the guidance, linked from this repo's README.md, on How to Create a Good Pull Request.

Following the guidance & completing the default template is required for all PRs to be reviewed in this repo.

@andrewconnell andrewconnell added Needs: Author Feedback Awaiting response from the original poster of the issue. Marked as stale if no activity for 7 days. Needs: Context Detail ❓ Need more context detail around the issue. pr:do-not-merge Pull request not ready to be merged type:incomplete-submission The submitted issue/PR is incomplete. New issues have sections you must address. and removed pr:in-review Actively reviewing pull request labels Feb 10, 2025
Copy link
Contributor

Learn Build status updates of commit 082ea05:

⚠️ Validation status: warnings

File Status Preview URL Details
docs/embedded/concepts/app-concepts/containertypes.md ⚠️Warning View Details
docs/embedded/concepts/admin-exp/consuming-tenant-admin/ctaUX.md 💡Suggestion View Details
docs/embedded/images/addowners-1.png ✅Succeeded View
docs/embedded/images/addowners-2.png ✅Succeeded View
docs/embedded/images/Filter.png ✅Succeeded View
docs/embedded/images/Reassignuser.png ✅Succeeded View
docs/embedded/images/Removeuser.png ✅Succeeded View
docs/embedded/images/Sensitivitylabel.png ✅Succeeded View
docs/embedded/images/Sorting.png ✅Succeeded View

docs/embedded/concepts/app-concepts/containertypes.md

  • Line 47, Column 194: [Warning: file-not-found - See documentation] Invalid file link: './New-SPOContainerType.md'.
  • Line 47, Column 357: [Warning: file-not-found - See documentation] Invalid file link: './Add-SPOContainerTypeBilling.md'.

docs/embedded/concepts/admin-exp/consuming-tenant-admin/ctaUX.md

  • Line 177, Column 1: [Suggestion: duplicate-alt-text - See documentation] Alt text 'SPE Container owner' is duplicated. Within an article, alt text must be unique.
  • Line 179, Column 1: [Suggestion: duplicate-alt-text - See documentation] Alt text 'SPE Container owner' is duplicated. Within an article, alt text must be unique.

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:

Copy link
Contributor

Learn Build status updates of commit 156457f:

⚠️ Validation status: warnings

File Status Preview URL Details
docs/embedded/concepts/app-concepts/containertypes.md ⚠️Warning View Details
docs/embedded/concepts/admin-exp/consuming-tenant-admin/ctaUX.md 💡Suggestion View Details
docs/embedded/images/addowners-1.png ✅Succeeded View
docs/embedded/images/addowners-2.png ✅Succeeded View
docs/embedded/images/Filter.png ✅Succeeded View
docs/embedded/images/Reassignuser.png ✅Succeeded View
docs/embedded/images/Removeuser.png ✅Succeeded View
docs/embedded/images/Sensitivitylabel.png ✅Succeeded View
docs/embedded/images/Sorting.png ✅Succeeded View

docs/embedded/concepts/app-concepts/containertypes.md

  • Line 47, Column 194: [Warning: file-not-found - See documentation] Invalid file link: './New-SPOContainerType.md'.
  • Line 47, Column 357: [Warning: file-not-found - See documentation] Invalid file link: './Add-SPOContainerTypeBilling.md'.

docs/embedded/concepts/admin-exp/consuming-tenant-admin/ctaUX.md

  • Line 177, Column 1: [Suggestion: duplicate-alt-text - See documentation] Alt text 'SPE Container owner' is duplicated. Within an article, alt text must be unique.
  • Line 179, Column 1: [Suggestion: duplicate-alt-text - See documentation] Alt text 'SPE Container owner' is duplicated. Within an article, alt text must be unique.

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:

@ShreyasSar26 ShreyasSar26 marked this pull request as draft February 17, 2025 04:37
@ShreyasSar26 ShreyasSar26 marked this pull request as ready for review February 17, 2025 05:32
Copy link
Contributor

Learn Build status updates of commit 1067530:

💡 Validation status: suggestions

File Status Preview URL Details
docs/embedded/concepts/admin-exp/consuming-tenant-admin/ctaUX.md 💡Suggestion View Details
docs/embedded/concepts/app-concepts/containertypes.md ✅Succeeded View
docs/embedded/images/addowners-1.png ✅Succeeded View
docs/embedded/images/addowners-2.png ✅Succeeded View
docs/embedded/images/Filter.png ✅Succeeded View
docs/embedded/images/Reassignuser.png ✅Succeeded View
docs/embedded/images/Removeuser.png ✅Succeeded View
docs/embedded/images/Sensitivitylabel.png ✅Succeeded View
docs/embedded/images/Sorting.png ✅Succeeded View

docs/embedded/concepts/admin-exp/consuming-tenant-admin/ctaUX.md

  • Line 177, Column 1: [Suggestion: duplicate-alt-text - See documentation] Alt text 'SPE Container owner' is duplicated. Within an article, alt text must be unique.
  • Line 179, Column 1: [Suggestion: duplicate-alt-text - See documentation] Alt text 'SPE Container owner' is duplicated. Within an article, alt text must be unique.

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:

Copy link
Contributor

Learn Build status updates of commit 897d486:

✅ Validation status: passed

File Status Preview URL Details
docs/embedded/concepts/admin-exp/consuming-tenant-admin/ctaUX.md ✅Succeeded View
docs/embedded/concepts/app-concepts/containertypes.md ✅Succeeded View
docs/embedded/images/addowners-1.png ✅Succeeded View
docs/embedded/images/addowners-2.png ✅Succeeded View
docs/embedded/images/Filter.png ✅Succeeded View
docs/embedded/images/Reassignuser.png ✅Succeeded View
docs/embedded/images/Removeuser.png ✅Succeeded View
docs/embedded/images/Sensitivitylabel.png ✅Succeeded View
docs/embedded/images/Sorting.png ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

@andrewconnell
Copy link
Collaborator

@ShreyasSar26 this PR has conflicts that must be resolved before it can be reviewed. Changing this PR status to DRAFT...

Once the conflicts have been resolved, flip the status back to OPEN to trigger a review to get this PR merged.

@andrewconnell andrewconnell marked this pull request as draft February 19, 2025 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs: Author Feedback Awaiting response from the original poster of the issue. Marked as stale if no activity for 7 days. Needs: Context Detail ❓ Need more context detail around the issue. pr:do-not-merge Pull request not ready to be merged type:incomplete-submission The submitted issue/PR is incomplete. New issues have sections you must address.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants