Skip to content

[CS2103-F15-4] TAssist#95

Open
ChinZJ wants to merge 674 commits into
nus-cs2103-AY2425S2:masterfrom
AY2425S2-CS2103-F15-4:master
Open

[CS2103-F15-4] TAssist#95
ChinZJ wants to merge 674 commits into
nus-cs2103-AY2425S2:masterfrom
AY2425S2-CS2103-F15-4:master

Conversation

@ChinZJ
Copy link
Copy Markdown

@ChinZJ ChinZJ commented Feb 23, 2025

CS2106 TAs struggle to efficiently track progress and access student information across multiple tutorial groups. The lack of a centralized system makes management and follow-ups tedious. TAssist provides a keyboard-driven platform that streamlines student organization into meaningful groups, enabling access to key details anytime, anywhere.

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 23, 2025

Codecov Report

❌ Patch coverage is 62.75304% with 184 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/main/java/seedu/tassist/ui/PersonCard.java 0.00% 57 Missing ⚠️
...u/tassist/model/person/PersonMatchesPredicate.java 4.34% 11 Missing and 11 partials ⚠️
...eedu/tassist/logic/parser/SearchCommandParser.java 17.39% 9 Missing and 10 partials ⚠️
src/main/java/seedu/tassist/model/Operations.java 65.90% 3 Missing and 12 partials ⚠️
...rc/main/java/seedu/tassist/ui/PersonListPanel.java 0.00% 12 Missing ⚠️
src/main/java/seedu/tassist/ui/MainWindow.java 0.00% 10 Missing ⚠️
.../main/java/seedu/tassist/commons/util/CsvUtil.java 18.18% 9 Missing ⚠️
...java/seedu/tassist/logic/commands/EditCommand.java 66.66% 0 Missing and 8 partials ⚠️
.../seedu/tassist/logic/parser/AddressBookParser.java 62.50% 6 Missing ⚠️
...in/java/seedu/tassist/logic/parser/ParserUtil.java 86.48% 3 Missing and 2 partials ⚠️
... and 10 more
Files with missing lines Coverage Δ Complexity Δ
...n/java/seedu/tassist/commons/core/GuiSettings.java 92.30% <ø> (ø) 12.00 <0.00> (ø)
...rc/main/java/seedu/tassist/logic/LogicManager.java 42.55% <ø> (ø) 3.00 <0.00> (ø)
src/main/java/seedu/tassist/logic/Messages.java 61.11% <100.00%> (-2.69%) 10.00 <0.00> (ø)
.../java/seedu/tassist/logic/commands/AddCommand.java 100.00% <ø> (ø) 8.00 <0.00> (ø)
...va/seedu/tassist/logic/commands/CommandResult.java 96.00% <100.00%> (-4.00%) 14.00 <4.00> (+1.00) ⬇️
...java/seedu/tassist/logic/commands/ExitCommand.java 100.00% <ø> (ø) 2.00 <0.00> (ø)
...java/seedu/tassist/logic/commands/HelpCommand.java 100.00% <ø> (ø) 2.00 <0.00> (ø)
.../seedu/tassist/logic/commands/LoadDataCommand.java 21.42% <100.00%> (+1.91%) 3.00 <1.00> (ø)
...java/seedu/tassist/logic/commands/RedoCommand.java 37.50% <100.00%> (ø) 2.00 <0.00> (ø)
...va/seedu/tassist/logic/commands/SearchCommand.java 100.00% <100.00%> (ø) 8.00 <1.00> (ø)
... and 32 more

... and 1 file with indirect coverage changes

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

@ChinZJ ChinZJ changed the title TAssist [CS2103-F15-4] TAssist Mar 14, 2025
lilyium added a commit to kerktaiheng/tp that referenced this pull request Mar 27, 2025
…425S2#96)

Add tests for Meeting commands in the logic component

Let's add tests for 
* AddMeetingCommand
* DeleteMeetingCommand
* AddMeetingCommandParser
* DeleteMeetingCommandParser

Note: Tests related to Meeting in the model and storage component have yet to be added
ChinZJ and others added 27 commits March 29, 2025 00:51
DevGuide could have used links as recommended.
Also, noted that "Tutorial att" label should not be shown
when there is no tutorial group.

In fact, the whole attendances pane should be
removed if there is no tutorial group.

Let's
* Update DG to show better practices
* Update PersonCard.java to support removal of attendances pane
Resolve bugs relating to MarkAttendanceCommand and Parser
Fix add command and UG bugs
Tag ordering is different on the displayed attributes compared to the tags displayed on the GUI.

Let's
* Sort the tags before displaying them
User Guide is outdated in terms of the marking of attendance.

Let's
* Add detailed descriptions of its behaviour to keep in line wiht the
  actual behaviour of the command
There is some outdated info on these guides.

Let's
* Update these documentation
# Conflicts:
#	src/main/java/seedu/tassist/logic/Messages.java
UML Diagrams still seem to be lacking.

Let's
* Update the UML Diagrams to make sure they are up-to-date
hsinyilow and others added 30 commits April 8, 2025 11:19
# Conflicts:
#	docs/DeveloperGuide.md
#	docs/UserGuide.md
# Conflicts:
#	docs/DeveloperGuide.md
UserGuide and DevGuide have some typos.

Let's
* Resolve these typos.
Use cases in DG are lacking.

Let's
* Add some use cases to DG
DG tabs are not consistent.

Let's
* Update DG tabs
Typos in DG.

Let's
* Update these
Lines are clumping in DG.

Let's
* Add lines to segregate them
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.

6 participants