Skip to content

OKTA-926630 - Update role assignment concept #5422

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.

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

brianduffield-okta
Copy link
Contributor

Description:

  • What's changed? Removed static standard role tables from this Concept and referenced the OAS3 site with this content
  • Is this PR related to a Monolith release? n/a

Resolves:

@acrolinxdevdocs-okta
Copy link
Contributor

Acrolinx score

A minimum Acrolinx Score of 80 is required. The total score is an average of the subscores.

Select Total score to review the Acrolinx scorecard for your article. Try to increase your individual scores, for example: Correctness. Your content will be clearer and more consistent.

Article Total score
Required:80
Word and phrases
(Brand, terms)
Preferred: 80
Correctness
(Spelling, grammar)
Preferred: 80
Clarity
(Readability)
Preferred: 80
Inclusive language
(+ accesibility)
Preferred: 80
packages/@okta/vuepress-site/docs/concepts/role-assignment/index.md 87 59 100 74 100

@eng-info-dev-github-bot
Copy link

Netlify Preview URL for the changes: https://preview-5422--reverent-murdock-829d24.netlify.app

Copy link
Contributor

@vanngo-okta vanngo-okta left a comment

Choose a reason for hiding this comment

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

The updates look good to me! 👍

There are 2 other references in this doc that have potentially outdated lists - maybe we can update the content or link it out to the openapi/okta-management/guides/roles file too?
image
image

Maybe this is outside of the scope of this ticket? It can be addressed in another task.

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

Successfully merging this pull request may close these issues.

4 participants