Skip to content
This repository has been archived by the owner on Dec 27, 2023. It is now read-only.

Update my PPP #205

Merged

Conversation

rayshawntan
Copy link

Update my PPP draft. Not fully complete yet, but to be released to meet tP dashboard deadline.

@rayshawntan rayshawntan added this to the v1.4 milestone Nov 10, 2023
Copy link

codecov bot commented Nov 10, 2023

Codecov Report

❗ No coverage uploaded for pull request base (master@bfd3808). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #205   +/-   ##
=========================================
  Coverage          ?   72.40%           
  Complexity        ?      439           
=========================================
  Files             ?       76           
  Lines             ?     1482           
  Branches          ?      144           
=========================================
  Hits              ?     1073           
  Misses            ?      367           
  Partials          ?       42           


## Contributions to the user guide (extracts)
### Contributions beyond the project team
* During PE-D, I have put in efforts to [find bugs](https://github.com/AY2324S1-CS2103T-T09-4/tp/issues?q=is%3Aissue+%22%5BPE-D%5D%5BTester+D%5D%22) that do not correspond to their UG.

Choose a reason for hiding this comment

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

Potential typo in "efforts"?

### Contributions to the UG

* Update implementations of `add` and `edit` features

Choose a reason for hiding this comment

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

There may be a bit of inconsistency in the pointers that do or don't end with fullstops.

@Cloud7050 Cloud7050 mentioned this pull request Nov 10, 2023
4 tasks
Copy link

@bwangpj bwangpj left a comment

Choose a reason for hiding this comment

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

@rayshawntan you can merge whenever you are ready

@rayshawntan rayshawntan merged commit c2c3678 into AY2324S1-CS2103-W14-3:master Nov 10, 2023
3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants