Skip to content

[CS2103-F15-1] TAskbook#111

Open
AnAfterthought wants to merge 808 commits into
nus-cs2103-AY2425S2:masterfrom
AY2425S2-CS2103-F15-1:master
Open

[CS2103-F15-1] TAskbook#111
AnAfterthought wants to merge 808 commits into
nus-cs2103-AY2425S2:masterfrom
AY2425S2-CS2103-F15-1:master

Conversation

@AnAfterthought
Copy link
Copy Markdown

TAskbook is designed to help Teaching Assistants track their tutorials and students. It is designed for CLI users so that they can manage records quickly with commands.

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 28, 2025

Codecov Report

❌ Patch coverage is 62.81896% with 306 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/main/java/seedu/address/model/AddressBook.java 70.83% 46 Missing and 24 partials ⚠️
...edu/address/ui/submission/SubmissionListPanel.java 0.00% 34 Missing ⚠️
...va/seedu/address/ui/submission/SubmissionCard.java 0.00% 24 Missing ⚠️
...u/address/logic/commands/SetSubmissionCommand.java 0.00% 17 Missing ⚠️
...gic/commands/DeleteStudentFromTutorialCommand.java 23.52% 9 Missing and 4 partials ⚠️
.../address/logic/commands/ListSubmissionCommand.java 0.00% 13 Missing ⚠️
...ss/logic/commands/AddStudentToTutorialCommand.java 29.41% 8 Missing and 4 partials ⚠️
...u/address/logic/commands/AddAssignmentCommand.java 61.53% 8 Missing and 2 partials ⚠️
...dress/logic/parser/SetSubmissionCommandParser.java 0.00% 8 Missing ⚠️
.../address/logic/commands/MarkAttendanceCommand.java 74.07% 3 Missing and 4 partials ⚠️
... and 38 more
Files with missing lines Coverage Δ Complexity Δ
...n/java/seedu/address/commons/core/index/Index.java 100.00% <100.00%> (ø) 12.00 <1.00> (+1.00)
src/main/java/seedu/address/logic/Messages.java 81.81% <100.00%> (-5.69%) 3.00 <0.00> (ø)
...u/address/logic/commands/export/ExportCommand.java 0.00% <ø> (ø) 0.00 <0.00> (?)
.../logic/commands/export/ExportTutorialsCommand.java 0.00% <ø> (ø) 0.00 <0.00> (?)
...a/seedu/address/logic/parser/AddCommandParser.java 100.00% <100.00%> (ø) 4.00 <0.00> (-1.00)
...address/logic/parser/AddTutorialCommandParser.java 100.00% <100.00%> (ø) 3.00 <0.00> (?)
...a/seedu/address/logic/parser/AttendanceParser.java 100.00% <100.00%> (ø) 3.00 <0.00> (?)
...ain/java/seedu/address/logic/parser/CliSyntax.java 93.75% <100.00%> (+10.41%) 1.00 <0.00> (ø)
...ress/logic/parser/DeleteTutorialCommandParser.java 100.00% <100.00%> (ø) 3.00 <0.00> (?)
.../seedu/address/logic/parser/EditCommandParser.java 68.75% <ø> (-23.85%) 8.00 <0.00> (-3.00)
... and 75 more

... and 4 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@nguyenvukhang nguyenvukhang force-pushed the master branch 5 times, most recently from e554c3a to ee77fd8 Compare March 4, 2025 14:52
meloppeitreet and others added 30 commits April 7, 2025 22:39
Fixes #310.
Developer Guide is modified, as the behaviour is otherwise correct.
Index 0 is ALWAYS an invalid index as it is not a positive integer (hence invalid command format).
Index 999 can be a valid index (but will throw invalid index if it is out of bounds in current scenario).
Student description not saved to storage
Edit Command Sequence Diagram a bit too small
…-message

Tutorials in student description after editing not separated clearly
Add planned enhancement for attendance feature
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.

5 participants