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

Update optional-claims-reference.md to mention the optional value on idtype #578

Closed
wants to merge 2 commits into from

Conversation

jmprieur
Copy link
Contributor

@jmprieur jmprieur commented Mar 28, 2024

@SHERMANOUKO this will also need to be propagated to 1P, ?What is your preference? shall I do another PR?

Copy link
Contributor

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

Copy link
Contributor

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

Copy link
Contributor

Learn Build status updates of commit e7613df:

✅ Validation status: passed

File Status Preview URL Details
docs/identity-platform/optional-claims-reference.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit 36b7f5a:

✅ Validation status: passed

File Status Preview URL Details
docs/identity-platform/optional-claims-reference.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@jmprieur
Copy link
Contributor Author

#sign-off

Copy link
Contributor

Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @cilwerner

@Court72
Copy link
Contributor

Court72 commented Mar 28, 2024

Hi @jmprieur - This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide. Would you make this content update in the private repo instead? Thank you! We'll close this PR.

@Court72 Court72 closed this Mar 28, 2024
@cilwerner
Copy link
Contributor

@jmprieur, I've fixed this!

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.

4 participants