Conversation
|
I just pushed the changes we talked about yesterday! I am ready for review. |
validbeck
left a comment
There was a problem hiding this comment.
- I think you missed editing
_quarto.ymlproperly OR you forgot to includeuser-setup.qmdin your push, as this is what I get after a render (where did all of our articles go??):
| No user management in nav | Broken link to user setup |
|---|---|
 |
 |
- I would change the title of that section to either
SETUPorCONFIGURATIONas well as previously discussed. I prefer the latter, as it covers both initial and tweaks. - Please do a sweep of these articles for margin content convention that was set up recently! Thank you 💖
You're right, I did not push the I swept for the margin convention, and there's no links to begin with, so yay! Just the log in to platform UI which is all good. Thanks for the review, let me know if you can see the changes now! |
|
Whoo, I see the files now! Section
View your profile
Yep, I'm telling you that there are some places you should add some, based on how they were added in other places. ;) |
|
I did some meditation and edited the CTA. I feel like it has more life in it now: |
There was a problem hiding this comment.
I did some meditation and edited the CTA. I feel like it has more life in it now
Nice! This is much better.
View your profile
- You're still just directing users to a page, instead of explaining the context of what they can do and how to do it. I've made the changes and pushed them, so all you need to do is pull them down and review them, but please re-render the site and test the static files and upload them before merging this request.
- For this one, you're not really referring to the right onboarding, so the margin link isn't relevant.
- I adjusted the "What's next" links too, as the ones you had before were a bit too specific.
For the future, really think about granular actions from the perspective of someone who has never used the interface before, and make sure you stress-test the functionality you're writing about to catch any edge-cases too (like the restart option!).
| View your profile updates | Onboarding wizard |
|---|---|
 |
 |
Set up your organization
- You missed some margin links on this page! No biggie, but it is part of this section.
…o noosheenvazinkhoo/sc-5077/review-and-restructure-articles-in-the-onboarding
Internal Notes for Reviewers
I tested the steps for
Manage groupsandManage usersagainst the UI, and made changes accordingly. Also, some of the formatting wasn't following the style guide, so I made edits there.Manage users:
Manage groups:
I moved two sections (User groups & Add or remove group members) from
Manage userstoManage groups. User groups had the same information as another section inManage groups, so I consolodated the articles.I'm testing making
View your profileits own page. I would like to add a screenshot that doesn't leak my email but still looks professional.Lastly, I changed the wording from
OnboardingtoSetup. These articles have sections that are accessed for onboarding, and sections for setup and maintenance. By switching toSetup, it still embodiesOnboarding, but also includes use cases for configuring stuff after onboarding.External Release Notes