Skip to content

Integrate OIDC into the prerequistes section#50

Merged
Maksimiliana merged 8 commits intomainfrom
PERFX-8866
Apr 2, 2026
Merged

Integrate OIDC into the prerequistes section#50
Maksimiliana merged 8 commits intomainfrom
PERFX-8866

Conversation

@Yusuf-Uzun
Copy link
Copy Markdown
Contributor

Integrate OIDC as a single sign-on method into the prerequistes section

@Maksimiliana Maksimiliana added contribution Valuable Contribution size/m Requires medium effort to get changed. type/content-gaps Something essential is missing in the documentation. type/content-structure The documentation structure needs to be improved. ua-review Under review by the UA team. labels Mar 23, 2026
Copy link
Copy Markdown
Contributor

@Hariharan-Gandhi Hariharan-Gandhi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Improvement: (both in SAML and OIDC)

Required column shall contain either Yes or Optional

SAML Section:
Conditionally with a statement Required if enabled is set to true. sounds superfluous.

OIDC Section:
same comment.
Required should be Yes or Required uniform. We can decide on one.

We can remove Required if enabled is set to true. from all required fields and make a single entry under enabled

@Maksimiliana Maksimiliana added size/l Requires large effort to get changed. and removed size/m Requires medium effort to get changed. labels Mar 30, 2026
@Maksimiliana Maksimiliana merged commit 59045c9 into main Apr 2, 2026
3 checks passed
@Maksimiliana Maksimiliana deleted the PERFX-8866 branch April 2, 2026 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contribution Valuable Contribution size/l Requires large effort to get changed. type/content-gaps Something essential is missing in the documentation. type/content-structure The documentation structure needs to be improved. ua-review Under review by the UA team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants