Skip to content

[do not merge] Adds guide for OAuth2 plugin, #PG-4998#870

Open
AltamashShaikh wants to merge 6 commits intolivefrom
PG-4998-oauth-guide
Open

[do not merge] Adds guide for OAuth2 plugin, #PG-4998#870
AltamashShaikh wants to merge 6 commits intolivefrom
PG-4998-oauth-guide

Conversation

@AltamashShaikh
Copy link
Copy Markdown
Contributor

@AltamashShaikh AltamashShaikh commented Mar 19, 2026

Description

Adds guide for OAuth2 plugin, should be merged after we release a version of plugin

Fixes: #PG-4998

Checklist

  • [✔] I have understood, reviewed, and tested all AI outputs before use
  • [✔] All AI instructions respect security, IP, and privacy rules

Review

@AltamashShaikh AltamashShaikh changed the title Adds guide for OAuth2 plugin, #PG-4998 [do not merge] Adds guide for OAuth2 plugin, #PG-4998 Mar 19, 2026
Copy link
Copy Markdown
Contributor

@lachiebol lachiebol left a comment

Choose a reason for hiding this comment

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

Looks good to me 👍 just a couple comments

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is this OK to update?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

my bad, will fix this one

Copy link
Copy Markdown
Contributor

@lachiebol lachiebol left a comment

Choose a reason for hiding this comment

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

LGTM 👍 left one optional thing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants