Skip to content

[CS2103-W11-4] TutorTrack#123

Open
Cheeden wants to merge 585 commits into
nus-cs2103-AY2526S1:masterfrom
AY2526S1-CS2103T-W11-4:master
Open

[CS2103-W11-4] TutorTrack#123
Cheeden wants to merge 585 commits into
nus-cs2103-AY2526S1:masterfrom
AY2526S1-CS2103T-W11-4:master

Conversation

@Cheeden

@Cheeden Cheeden commented Sep 21, 2025

Copy link
Copy Markdown

Helps tutors centralize and quickly access student and parent contact information with key learning notes, reducing time spent searching and allowing them to focus more on lesson preparation.

Cheeden pushed a commit to Cheeden/tp that referenced this pull request Oct 27, 2025
jooh6969 pushed a commit to jooh6969/tp that referenced this pull request Oct 28, 2025
Make help window point to ClubHub userguide
yatihe01 and others added 26 commits October 30, 2025 16:56
…-contacts

Fix userguide description for contacts
Currently, if the user uses find command of find d/Tomorrow, where
tomorrow is an invalid day. The error message would be Contact list
is unchanged: No students match your search criteria. This error
message can be clearer to let the user know why it is wrong.

- Add MESSAGE_INVALID_DAY constant to Messages.java for consistent error
messaging
- Implement ParserUtil.parseDay() to validate day of week input
using regex
- Update FindCommandParser to validate day parameter before creating
predicate
- Add comprehensive test cases for invalid day inputs (abbreviations,
typos, invalid names)
- Update User Guide with validation requirements and error examples
- Update Developer Guide with implementation details and validation
logic

The find d/ command now only accepts full day names (Monday-Sunday) and
rejects abbreviations (e.g., "Mon"), typos (e.g., "Mondayy"), and invalid
inputs like "Tomorrow" or "1" with a clear error message.
Refine UG by removing duplicated descriptions and add glossary section
nicksonho and others added 30 commits November 4, 2025 11:32
Add tests for edit plan & progress command parser
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