Skip to content
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 AboutUs, PPP, and UG #31

Merged
merged 10 commits into from
Oct 4, 2023

Conversation

brandon-nam
Copy link
Collaborator

No description provided.

@codecov
Copy link

codecov bot commented Oct 4, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (624133d) 75.26% compared to head (957c05f) 75.26%.

❗ Current head 957c05f differs from pull request most recent head dd53d94. Consider uploading reports for the commit dd53d94 to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #31   +/-   ##
=========================================
  Coverage     75.26%   75.26%           
  Complexity      419      419           
=========================================
  Files            71       71           
  Lines          1338     1338           
  Branches        126      126           
=========================================
  Hits           1007     1007           
  Misses          301      301           
  Partials         30       30           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Collaborator

@H1410101 H1410101 left a comment

Choose a reason for hiding this comment

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

Commented some suggestions, should be minor and can be merged after.

docs/UserGuide.md Outdated Show resolved Hide resolved
Comment on lines 130 to 131
2. Type `view INDEX` to enter the profile page of the fosterer with the corresponding `INDEX` from the `list` command.
3. Type `edit` to edit the fosterer's details (alternatively, you can skip step 2 by typing `edit INDEX` command.)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Think it makes sense for view to be an alias of edit (same functionality)?

docs/UserGuide.md Outdated Show resolved Hide resolved
3. Type `edit` to edit the fosterer's details (alternatively, you can skip step 2 by typing `edit INDEX` command.)
4. In the command line, type the name of the field you want to edit and press enter. This converts a plain text into a
text field with the detail.
5. Edit the content of the detail and press enter.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
5. Edit the content of the detail and press enter.
5. Edit the content of the detail and press enter to confirm the changes, or the esc key to cancel them.

docs/UserGuide.md Outdated Show resolved Hide resolved
@H1410101 H1410101 merged commit 17bf103 into AY2324S1-CS2103T-T13-4:master Oct 4, 2023
3 checks passed
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.

3 participants