Skip to content

Conversation

@freakwriter
Copy link

KnowledgeOwl's navigation menus and UI have changed significantly; updated text and screenshots to reflect those changes.

Remove old screenshots and replace with updated ones.
Update to reflect current UI steps.
@freakwriter
Copy link
Author

@microsoft-github-policy-service agree company="KnowledgeOwl"

@prmerger-automator
Copy link
Contributor

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

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 17d4e83:

⚠️ Validation status: warnings

File Status Preview URL Details
docs/identity/saas-apps/knowledgeowl-tutorial.md ⚠️Warning Details
docs/identity/saas-apps/media/knowledgeowl-tutorial/idp-metadata.png 💡Suggestion Details
docs/identity/saas-apps/media/knowledgeowl-tutorial/saml-attribute-map-save.png ✅Succeeded
docs/identity/saas-apps/media/knowledgeowl-tutorial/saml-attribute-map.png ✅Succeeded
docs/identity/saas-apps/media/knowledgeowl-tutorial/security-sso-menu.png ✅Succeeded
docs/identity/saas-apps/media/knowledgeowl-tutorial/settings-sso-dropdown.png ✅Succeeded n/a (file deleted or renamed)
docs/identity/saas-apps/media/knowledgeowl-tutorial/sp-metadata.png ✅Succeeded
docs/identity/saas-apps/media/knowledgeowl-tutorial/sso-saml-settings.png ✅Succeeded
docs/identity/saas-apps/media/knowledgeowl-tutorial/sso-settings-direct-attribute-fields.png ✅Succeeded n/a (file deleted or renamed)
docs/identity/saas-apps/media/knowledgeowl-tutorial/sso-settings-direct-attribute-save.png ✅Succeeded n/a (file deleted or renamed)
docs/identity/saas-apps/media/knowledgeowl-tutorial/sso-settings-required-fields.png ✅Succeeded n/a (file deleted or renamed)
docs/identity/saas-apps/media/knowledgeowl-tutorial/sso-settings-saml-save.png ✅Succeeded n/a (file deleted or renamed)

docs/identity/saas-apps/knowledgeowl-tutorial.md

  • Line 140, Column 5: [Warning: file-not-found - See documentation] Invalid file link: './media/knowledgeowl-tutorial/settings-sso-dropdown.png'.

docs/identity/saas-apps/media/knowledgeowl-tutorial/idp-metadata.png

  • Line 0, Column 0: [Suggestion: image-name-incomplete - See documentation] The image name 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.

Helps if you reference the new screenshot.
Remove idp and make clearer, update reference in doc.
@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.
Changed Files This PR contains more than 10 changed files.
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).

@ktoliver ktoliver requested a review from Copilot November 25, 2025 23:14
Copilot finished reviewing on behalf of ktoliver November 25, 2025 23:16
@ktoliver
Copy link
Contributor

@nguhiu

Can you review the proposed changes?

It's a good idea to move the commit to the private repo to verify images and procedures on staging. Let us know if you'd like our help with that.

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

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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.

3 participants