Conversation
|
After looking through the other docs in this section, I realized that a lot of the steps in bullet points should be in their own section as numbered steps, like the one in the Manage roles guide. Should I make a new PR, or edit this story to include this restructuring? |
validbeck
left a comment
There was a problem hiding this comment.
Nice! The "Onboarding users" page looks great now. You can go ahead and render the files and send them up I think, before you merge the commit in (gives you and opportunity to check the major links in the site as well to make sure they work in the static files, catch any missed merge conflicts, etc.)
I realized that a lot of the steps in bullet points should be in their own section as numbered steps, like the one in the Manage roles guide. Should I make a new PR, or edit this story to include this restructuring?
I think you can make a new Story to review the guides in this section yes, as you are likely needing to retool Manage users into at least 2 new articles!
Internal Notes for Reviewers
There were a few instances in the Onboarding Users section that mention Group Admin or Superuser, so I replaced them with Customer Admin. Mentions of Group Admin were removed completely in the definitions section.
The following changes have already been merged, but relate to this story:
The Onboarding users page was pretty rough. I removed the Example section, and replaced it with a What's next section with CTA cards.
Sidenote: I tried to add only the changed files like Nik said, so let me know if the Git diff experience is better!
External Release Notes