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

Edit UG #117

Merged
merged 40 commits into from
Apr 2, 2024
Merged

Conversation

Rikko1204
Copy link

No description provided.

cheahTJ and others added 30 commits March 17, 2024 23:27
delete command had no functionality to delete via a group. Added this functionality so that it is easier to delete large groups of contacts at once if required.
…tiple-users

Group multiple users in single command
Change position of tag to align to the right side of the person card.
Added color background for tag ui
If there is no tag assign to the person, no tag ui will be displayed
…15-2/delete_feature

Able to delete in group for delete feature
cheahTJ and others added 10 commits March 27, 2024 19:29
* master:
  Update DeleteCommand.java
  resolve errors
  Update DeleteCommandParser.java
  documentationupdate
  Update DeleteCommandParser.java
  Update DeleteCommandParser.java
  Resolve review
  update testcases
  checkstyle
  Update DeleteCommand.java
  Update DeleteCommand.java
  update checkstyle
  Update DeleteCommandParser.java
  Update DeleteCommand.java
  modify-delete-command
…command

Fix missing messages after merge conflict
…15-2/add-group-command

Add group command
…15-2/ui-beautification

UI beautification
Copy link

codecov bot commented Apr 1, 2024

Codecov Report

Attention: Patch coverage is 72.18935% with 47 lines in your changes are missing coverage. Please review.

Project coverage is 75.99%. Comparing base (02d35e7) to head (77c0852).
Report is 7 commits behind head on documentation_1.3.

Files Patch % Lines
src/main/java/seedu/address/ui/PersonCard.java 0.00% 18 Missing ⚠️
...va/seedu/address/logic/commands/DeleteCommand.java 63.15% 10 Missing and 4 partials ⚠️
...seedu/address/logic/parser/GroupCommandParser.java 41.66% 2 Missing and 5 partials ⚠️
...ava/seedu/address/logic/commands/GroupCommand.java 92.10% 0 Missing and 3 partials ⚠️
...eedu/address/logic/parser/DeleteCommandParser.java 81.81% 1 Missing and 1 partial ⚠️
.../seedu/address/logic/parser/AddressBookParser.java 0.00% 1 Missing ⚠️
...a/seedu/address/logic/parser/PinCommandParser.java 87.50% 0 Missing and 1 partial ⚠️
...a/seedu/address/model/person/UniquePersonList.java 80.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@                   Coverage Diff                   @@
##             documentation_1.3     #117      +/-   ##
=======================================================
- Coverage                76.44%   75.99%   -0.46%     
- Complexity                 521      559      +38     
=======================================================
  Files                       82       84       +2     
  Lines                     1690     1816     +126     
  Branches                   169      196      +27     
=======================================================
+ Hits                      1292     1380      +88     
- Misses                     347      370      +23     
- Partials                    51       66      +15     

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

@cheahTJ cheahTJ merged commit a0a8149 into AY2324S2-CS2103T-T15-2:documentation_1.3 Apr 2, 2024
3 of 5 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.

4 participants