-
Notifications
You must be signed in to change notification settings - Fork 35
Update personas.md #572
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
Update personas.md #572
Conversation
Updating personas based on output of design clinic Jan - Mar 2025
✅ Deploy Preview for nebari-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
smeragoel
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cleaning up markdown, and introduction text.
Cleaning up text and markdown spacing
Adding dividers between the personas and removing extra emojis
|
This is ready for review! One change I’d like to flag (which I don't know how to do) is moving the |
|
The linter needs to be run locally to address the fixes, for some reason the --attempt-fixes is not working on CI. @smeragoel can you try runnning |
|
@viniciusdc: I'll push up a commit to fix the linting issues. :) @smeragoel: +1 to moving the page in a separate PR. |
Signed-off-by: Pavithra Eswaramoorthy <[email protected]>
Signed-off-by: Pavithra Eswaramoorthy <[email protected]>
|
Thanks for this PR, @smeragoel! 🎉 |
Updating personas based on output of design clinic Jan - Mar 2025
Reference Issues or PRs
What does this implement/fix?
Put a
xin the boxes that applyTesting
Documentation
Access-centered content checklist
Text styling
H1or#in markdown).Non-text content
Any other comments?