Skip to content

Conversation

hwan-britive
Copy link

We've updated the Britive documentation to reflect product updates.

sangeetab-britive and others added 11 commits September 16, 2025 16:25
New screenshots for Azure SSO and SCIM.
Updated Britive UI screenshots!
Updated with new Britive UI screenshots.
Updated to match the new Britive UI.
Updated to match the new Britive UI.
Updated tutorials to match the new Britive UI.
Copy link
Contributor

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

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

Copy link
Contributor

PRMerger Results

Issue Description
Image File This PR added or updated an image file(s).

@v-regandowner
Copy link
Contributor

@jeevansd - Can you review the proposed changes?

IMPORTANT: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

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

@jeevansd
Copy link

#sign-off

Copy link
Contributor

You typed hold-off, but the PR wasn't signed off, so nothing was changed.

Copy link
Contributor

PRMerger Results

Issue Description
Image File This PR added or updated an image file(s).

@hwan-britive
Copy link
Author

HI @ttorble @jeevansd, the latest commit addresses regarding the GUID. Is there anything else you need from me for the PR?

@v-dirichards
Copy link
Contributor

@jeevansd Would you add a comment to indicate what action to take on this PR? Thanks!

Copy link

@Copilot 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 updates the Britive documentation to reflect changes in the Britive user interface, ensuring the tutorial steps and screenshots match the current product experience.

  • Updated UI element names to match the new Britive interface (e.g., "ADD IDENTITY PROVIDER" → "Add Identity Provider")
  • Corrected grammatical errors and improved sentence structure throughout the documentation
  • Standardized capitalization and formatting of UI elements and button names

Reviewed Changes

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

File Description
docs/identity/saas-apps/britive-tutorial.md Updated UI element names, fixed grammar, and improved readability for the main tutorial
docs/identity/saas-apps/britive-provisioning-tutorial.md Updated UI element names and corrected minor grammatical issues in the provisioning tutorial

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

Copy link
Contributor

PRMerger Results

Issue Description
Image File This PR added or updated an image file(s).

Copy link
Contributor

PRMerger Results

Issue Description
Image File This PR added or updated an image file(s).

@jeevansd
Copy link

reviewed and approved. #sign-off

@v-dirichards
Copy link
Contributor

@hwan-britive @jeevansd

🚫 Blocking issues that must be resolved before this PR can be merged:

docs/identity/saas-apps/media/britive-tutorial/configuration.png

  • Could you replace the highlighted Entity ID value with one of these approved fictitious values:
    • 00aa00aa-bb11-cc22-dd33-44ee44ee44ee
    • 11bb11bb-cc22-dd33-ee44-55ff55ff55ff
    • 22cc22cc-dd33-ee44-ff55-66aa66aa66aa
    • 33dd33dd-ee44-ff55-aa66-77bb77bb77bb
    • 44ee44ee-ff55-aa66-bb77-88cc88cc88cc
    • 55ff55ff-aa66-bb77-cc88-99dd99dd99dd
    • 66aa66aa-bb77-cc88-dd99-00ee00ee00ee
image

After you resolve the blocking issues, sign off again to trigger another review.

The blocked label is used to track this PR. Please don't remove the label.

#hold-off
#label:"blocked-1"

@hwan-britive
Copy link
Author

Hi @v-dirichards, we've made the change you request. Thank you.

Copy link
Contributor

PRMerger Results

Issue Description
Commits Count This PR contains more than 15 commits.
Image File This PR added or updated an image file(s).

@ShannonLeavitt
Copy link
Contributor

@hwan-britive - Thanks for cleaning up those IDs. I still see one ID that needs to be replaced:

image

@hwan-britive
Copy link
Author

Hi @ShannonLeavitt, this is the first UUID in a list posted by @v-dirichards

Screenshot 2025-10-03 at 1 29 08 PM

Please let us know if we should do something different.

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.

8 participants