Skip to content

Update ctaUX.md #9969

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

Merged
merged 1 commit into from
Oct 16, 2024
Merged

Update ctaUX.md #9969

merged 1 commit into from
Oct 16, 2024

Conversation

AlexanderHolmeset
Copy link
Contributor

@AlexanderHolmeset AlexanderHolmeset commented Oct 14, 2024

Link was pointing to wrong repository. But maybe it should link directly to the learn page? https://learn.microsoft.com/en-us/sharepoint/dev/embedded/concepts/admin-exp/adminrole

Category

  • Content fix
  • New article

Related issues

  • fixes #issuenumber
  • partially #issuenumber
  • mentioned in #issuenumber

What's in this Pull Request?

Submission guidelines

  • !!IMPORTANT!! - All submissions must complete the baseline sections included in this template. Ignoring or deleting this template may result in closing the issue with the label type:incomplete-submission.
  • Follow our guidance on How To Create Good Pull Requests.
  • Target the main branch of this repo.
  • When changing a page, ensure you update the ms.date front matter wih the current date in the format MM/DD/YYYY.
  • Review all build checks and address the automated errors, warnings, and suggestions.
  • NOTE: The live site is based on the live branch. Site owners periodically refresh live branch from the main branch so merged PRs won't immediately appear on the live site. Please be patient to see your changes appear on the live site.

DELETE THIS SECTION BEFORE SUBMITTING

Link was pointing to wrong repository. But maybe it should link directly to the learn page?
https://learn.microsoft.com/en-us/sharepoint/dev/embedded/concepts/admin-exp/adminrole
Copy link
Contributor

Learn Build status updates of commit 318129a:

✅ Validation status: passed

File Status Preview URL Details
docs/embedded/concepts/admin-exp/consuming-tenant-admin/ctaUX.md ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

@VesaJuvonen
Copy link
Contributor

Good catch - should be pointing actually to the https://learn.microsoft.com/en-us/sharepoint/dev/embedded/concepts/admin-exp/adminrole where the published doc is... will merge and will personally fix that to the end location. Thanks for pointing out the issue 🙏

@VesaJuvonen VesaJuvonen merged commit a4c514b into SharePoint:main Oct 16, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants