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.

Sign up for GitHub

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

Jump to bottom

update(platform): auth policy settings #114

Merged
merged 11 commits into from
May 14, 2024
Merged

update(platform): auth policy settings #114

merged 11 commits into from
May 14, 2024

Conversation

staceysalamon-aiven
Copy link
Contributor

Describe your changes

Update authentication policy doc for new UI language and settings.

Checklist

  • The first paragraph of the page is on one line.
  • The other lines have a line break at 90 characters.
  • I checked the output.
  • I applied the style guide.
  • My links start with /docs/.

@staceysalamon-aiven staceysalamon-aiven self-assigned this Mar 13, 2024
@cloudflare-workers-and-pages Cloudflare Workers and Pages
Copy link

cloudflare-workers-and-pages bot commented Mar 13, 2024 •
edited
Loading

Deploying aiven-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 7d728a1
Status: ✅  Deploy successful!
Preview URL: https://f9592e07.aiven-docs.pages.dev
Branch Preview URL: https://staceys-auth-policy-doc-913.aiven-docs.pages.dev

View logs

@staceysalamon-aiven staceysalamon-aiven marked this pull request as ready for review March 13, 2024 13:09
@staceysalamon-aiven staceysalamon-aiven added the review/SME An SME must review the PR. label Mar 13, 2024
@staceysalamon-aiven staceysalamon-aiven added review/TW A TW must review the PR. Comes after review/SME. and removed review/SME An SME must review the PR. labels Mar 15, 2024
@staceysalamon-aiven staceysalamon-aiven requested a review from a team March 15, 2024 13:09
ArthurFlag
ArthurFlag reviewed Mar 19, 2024
View reviewed changes
Copy link
Member

@ArthurFlag ArthurFlag left a comment

Choose a reason for hiding this comment

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

nitpick, the rest lgtm 👍

docs/platform/concepts/managed-users.md Outdated Show resolved Hide resolved
@staceysalamon-aiven @ArthurFlag
Update docs/platform/concepts/managed-users.md
9eefed7 
Co-authored-by: Arthur <arthur.flageul-marquez@aiven.io>
Signed-off-by: Stacey Salamon <111294980+staceysalamon-aiven@users.noreply.github.com>
@staceysalamon-aiven staceysalamon-aiven added review/awaitingRelease All reviews are completed. and removed review/TW A TW must review the PR. Comes after review/SME. labels Mar 19, 2024
staceysalamon-aiven and others added 2 commits April 11, 2024 11:47
@staceysalamon-aiven
update personal token details
a7ff979
@staceysalamon-aiven
Merge branch 'main' into staceys-auth-policy-doc-913
7d728a1 
Signed-off-by: Stacey Salamon <111294980+staceysalamon-aiven@users.noreply.github.com>
@staceysalamon-aiven staceysalamon-aiven merged commit 7a61ef2 into main May 14, 2024
4 checks passed
@staceysalamon-aiven staceysalamon-aiven deleted the staceys-auth-policy-doc-913 branch May 14, 2024 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ArthurFlag ArthurFlag ArthurFlag approved these changes

@rsalevsky rsalevsky Awaiting requested review from rsalevsky

Assignees

@staceysalamon-aiven staceysalamon-aiven

Labels
platform review/awaitingRelease All reviews are completed.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@staceysalamon-aiven @rsalevsky @ArthurFlag