AY2223S1-CS2103T-W08-4 / tp

AB3 for Team Project
https://ay2223s1-cs2103t-w08-4.github.io/tp/
MIT License
1 stars 5 forks source link

Fix to few bugs #246

Closed Bahamas20 closed 2 years ago

Bahamas20 commented 2 years ago

Fixed bugs:

207

209

228

176

195

200

codecov-commenter commented 2 years ago

Codecov Report

Merging #246 (ed2da72) into master (fee394e) will decrease coverage by 0.05%. The diff coverage is 50.00%.

@@             Coverage Diff              @@
##             master     #246      +/-   ##
============================================
- Coverage     68.72%   68.67%   -0.06%     
  Complexity      709      709              
============================================
  Files           112      112              
  Lines          2491     2493       +2     
  Branches        303      304       +1     
============================================
  Hits           1712     1712              
- Misses          668      669       +1     
- Partials        111      112       +1     
Impacted Files Coverage Δ
.../java/seedu/address/logic/commands/AddCommand.java 100.00% <ø> (ø)
...rc/main/java/seedu/address/model/person/Phone.java 90.00% <ø> (ø)
...edu/address/logic/parser/AddTaskCommandParser.java 61.11% <33.33%> (-7.64%) :arrow_down:
...a/seedu/address/logic/commands/AddItemCommand.java 88.23% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

weesiau commented 2 years ago

Resolves #206