-
Notifications
You must be signed in to change notification settings - Fork 0
Conversation
… 18-write-the-readme
WalkthroughAdds two "User Guide" links to README.md (with differing capitalization/paths) and introduces a new, detailed Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Free 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
Note 🎁 Summarized by CodeRabbit FreeYour organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login. Comment |
removed a note in text
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.
Nice!
Suggestions:
- Rename "Menu Bar" to "Side Bar"
- Move the UserGuide foler to the root folder
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.
Left some comments
…creenshots to public/img #18
…loyExplorer into 18-write-the-readme
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.
LGTM!
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.
LGTM!
Issue number
Closes 18
Description
Created the User Guide
Klein had to make the PR because of a bug, Alicia wrote the User Guide/ completed this issue
Testing steps
Does the User Guide show up
Check the links between the User Guide and README
Check that the screenshots show up
Screenshots (optional)
Summary by CodeRabbit