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: add org permissions and roles #532

Merged
merged 12 commits into from
Dec 13, 2024
Merged

update: add org permissions and roles #532

merged 12 commits into from
Dec 13, 2024

Conversation

staceysalamon-aiven
Copy link
Contributor

Describe your changes

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 Oct 29, 2024
@cloudflare-workers-and-pages Cloudflare Workers and Pages
Copy link

cloudflare-workers-and-pages bot commented Oct 29, 2024 •
edited
Loading

Deploying aiven-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 420135f
Status: ✅  Deploy successful!
Preview URL: https://890217e7.aiven-docs.pages.dev
Branch Preview URL: https://staceys-doc-1200.aiven-docs.pages.dev

View logs

@staceysalamon-aiven staceysalamon-aiven force-pushed the staceys-doc-1200 branch 2 times, most recently from 533afeb to 177d276 Compare October 31, 2024 10:28
@staceysalamon-aiven
change structure
5a0700d 
Create separate org and project sections with subsections
for roles and permissions in each.
@staceysalamon-aiven staceysalamon-aiven marked this pull request as ready for review December 9, 2024 14:22
@staceysalamon-aiven staceysalamon-aiven requested a review from a team as a code owner December 9, 2024 14:22
@staceysalamon-aiven
Copy link
Contributor Author

Preview: https://staceys-doc-1200.aiven-docs.pages.dev/docs/platform/concepts/permissions

@staceysalamon-aiven staceysalamon-aiven added the review/SME An SME must review the PR. label Dec 9, 2024
@github-actions github-actions bot added the tech Relates to technical updates of the docs platform. label Dec 11, 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 Dec 13, 2024
harshini-rangaswamy
harshini-rangaswamy approved these changes Dec 13, 2024
View reviewed changes
Copy link
Contributor

@harshini-rangaswamy harshini-rangaswamy 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! Noticed some minor typos.

@staceysalamon-aiven staceysalamon-aiven merged commit 43eb299 into main Dec 13, 2024
4 checks passed
@staceysalamon-aiven staceysalamon-aiven deleted the staceys-doc-1200 branch December 13, 2024 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@harshini-rangaswamy harshini-rangaswamy harshini-rangaswamy approved these changes

@rsalevsky rsalevsky rsalevsky approved these changes

Assignees

@staceysalamon-aiven staceysalamon-aiven

Labels
platform review/TW A TW must review the PR. Comes after review/SME. tech Relates to technical updates of the docs platform.
Projects
None yet
Milestone
No milestone
Development

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