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

[CS2103T-T13-1] Admin Book #79

Open
wants to merge 695 commits into
base: master
Choose a base branch
from

Conversation

taniatsq
Copy link

Admin Book helps tutors to keep track of student contact and other relevant details.

Copy link

codecov bot commented Feb 25, 2024

Codecov Report

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

Project coverage is 54.42%. Comparing base (b896eb0) to head (8978f27).

❗ Current head 8978f27 differs from pull request most recent head 2356fad. Consider uploading reports for the commit 2356fad to get more accurate results

Files Patch % Lines
.../address/logic/commands/EditAttendanceCommand.java 0.00% 97 Missing ⚠️
.../java/seedu/address/logic/commands/AddCommand.java 0.00% 76 Missing ⚠️
src/main/java/seedu/address/ui/PersonCard.java 0.00% 46 Missing ⚠️
...ess/logic/commands/AddAttendanceRecordCommand.java 21.56% 39 Missing and 1 partial ⚠️
.../logic/commands/DeleteAttendanceRecordCommand.java 59.34% 28 Missing and 9 partials ⚠️
...rc/main/java/seedu/address/model/ModelManager.java 55.69% 33 Missing and 2 partials ⚠️
...ress/logic/parser/EditAttendanceCommandParser.java 0.00% 29 Missing ⚠️
.../address/logic/commands/AddDescriptionCommand.java 0.00% 26 Missing ⚠️
...java/seedu/address/logic/commands/EditCommand.java 32.25% 19 Missing and 2 partials ⚠️
...in/java/seedu/address/logic/parser/ParserUtil.java 59.37% 12 Missing and 1 partial ⚠️
... and 33 more
Additional details and impacted files
@@              Coverage Diff              @@
##             master      #79       +/-   ##
=============================================
- Coverage     75.26%   54.42%   -20.84%     
- Complexity      419      522      +103     
=============================================
  Files            71      100       +29     
  Lines          1338     2352     +1014     
  Branches        126      259      +133     
=============================================
+ Hits           1007     1280      +273     
- Misses          301      994      +693     
- Partials         30       78       +48     

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

@taniatsq taniatsq changed the title Admin Book [CS2103-G16-1] Admin Book Feb 25, 2024
@taniatsq taniatsq changed the title [CS2103-G16-1] Admin Book [CS2103T-G16-1] Admin Book Feb 25, 2024
@taniatsq taniatsq changed the title [CS2103T-G16-1] Admin Book [CS2103T-T13-1] Admin Book Feb 25, 2024
taniatsq and others added 26 commits April 8, 2024 23:31
Make remove class hide all current students
Fix all find command related bugs
Update remove class command to hide filepath in UI (bottom left corner)
Update UG explaining behaviour of multiple search terms "find"
Update Add Command to throw error when trying to add before select
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