Skip to content
This repository has been archived by the owner on Jan 29, 2024. It is now read-only.

Add documentation for managed users #2147

Closed
wants to merge 5 commits into from

Conversation

staceysalamon-aiven
Copy link
Contributor

@staceysalamon-aiven staceysalamon-aiven commented Sep 15, 2023

What changed, and why it matters

Relies on #2339 (broken link error will be resolved when that is merged)

  • Added doc explaining managed users feature
  • Added instructions for setting org authentication policies

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Sep 15, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 820eea3
Status:🚫  Build failed.

View logs

Copy link

@rsalevsky rsalevsky left a comment

Choose a reason for hiding this comment

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

As discussed let's split this into two PRs managed users and auth policies. Let's also add more content of each auth policy option and the effect it has on the user.

@staceysalamon-aiven
Copy link
Contributor Author

Auth policies PR: #2164

Copy link

@rsalevsky rsalevsky left a comment

Choose a reason for hiding this comment

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

Please add the regular section that this feature is in preview.

docs/platform/concepts/managed-users.rst Outdated Show resolved Hide resolved
docs/platform/concepts/managed-users.rst Outdated Show resolved Hide resolved
@staceysalamon-aiven staceysalamon-aiven marked this pull request as ready for review December 8, 2023 11:40
@staceysalamon-aiven staceysalamon-aiven dismissed rsalevsky’s stale review December 8, 2023 11:41

Made requested changes

@staceysalamon-aiven
Copy link
Contributor Author

Replaced by #2341

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants