Skip to content

Specify the authentication methods we have available - #187

Merged
yuvipanda merged 2 commits into
2i2c-org:mainfrom
yuvipanda:auth
Jun 20, 2023
Merged

yuvipanda merged 2 commits into
2i2c-org:mainfrom
yuvipanda:auth

Conversation

@yuvipanda

Copy link
Copy Markdown
Member
  • Mention CILogon as primariy method
  • State that we try not to support other authentication providers, and will incur extra cost

- Mention CILogon as primariy method
- State that we try not to support other authentication providers,
  and will incur extra cost

@GeorgianaElena GeorgianaElena left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you @yuvipanda! 🚀

Comment thread admin/howto/manage-users.md Outdated
Co-authored-by: Georgiana <georgiana.dolocan@gmail.com>
@choldgraf choldgraf changed the title Fix authentication methods we have available Specify the authentication methods we have available Jun 13, 2023

@choldgraf choldgraf left a comment •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is this PR waiting on somebody specific to approve before merging? I'm trying to understand why we have 2 approvals over 7 days but no merge.

Maybe this is indicating uncertainty about who has authority to merge PRs to docs? Are people waiting for me specifically?

Or maybe we are applying the same principle of "the PR author must be the one to merge" that we use in changing running cloud infrastructure?

@yuvipanda
yuvipanda merged commit bed567f into 2i2c-org:main Jun 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants