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

chore: delete docs/identity/saas-apps/kallidus-tutorial.md #1330

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

chrisnottatkallidus
Copy link

This application has been removed from the app gallery and so the tutorial is no longer relevant and creates user confusion.

Copy link
Contributor

Learn Build status updates of commit 029f4e7:

⚠️ Validation status: warnings

File Status Preview URL Details
docs/identity/saas-apps/toc.yml ⚠️Warning Details
docs/identity/saas-apps/kallidus-tutorial.md ✅Succeeded n/a (file deleted or renamed)

docs/identity/saas-apps/toc.yml

  • Line 1555, Column 15: [Warning: file-not-found - See documentation] Invalid file link: 'kallidus-tutorial.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:

@ttorble
Copy link
Contributor

ttorble commented Jan 21, 2025

@chrisnottatkallidus Thank you for your contribution. Would you take a moment to sign the Contributor License Agreement (CLA)? After the CLA is signed, someone can review your pull request. Thanks!

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@Jak-MS
Copy link
Contributor

Jak-MS commented Jan 21, 2025

fyi @jeevansd @CelesteDG .

CLA not signed yet, but we can't delete the article in the public repo. Bears looking into and if the app no longer exists, maybe you can make a change in the private repo?

#assign: @jeevansd

@CelesteDG
Copy link
Collaborator

Thank you, @chrisnottatkallidus

Flagging @omondiatieno to verify in the App Gallery and confirm. Please confirm here.

@chrisnottatkallidus
Copy link
Author

chrisnottatkallidus commented Jan 23, 2025 via email

Copy link
Contributor

@chrisnottatkallidus : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ttorble
Copy link
Contributor

ttorble commented Jan 24, 2025

Thanks for signing the CLA @chrisnottatkallidus

Assigning @omondiatieno per @CelesteDG comment.

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