[CS2103T-F09-4] FitBook#210
Open
eltry42 wants to merge 481 commits into
Open
Conversation
Codecov Report❌ Patch coverage is 📢 Thoughts on this report? Let us know! |
- Add SortByPaidCommand class with sortbypaid command - Add SortByPaidCommandParser for parsing sortbypaid command - Add sortPersonListByPaid method to Model interface - Implement sorting logic in ModelManager, AddressBook, and UniquePersonList - Update AddressBookParser to handle sortbypaid command - Add comprehensive tests for SortByPaidCommand and SortByPaidCommandParser - Sort by paid status: unpaid clients first, paid clients second - Follows naming convention: sortby[field] for consistency with teammates
- Remove trailing whitespace from SortByPaidCommandParser - Remove unused imports from SortByPaidCommandTest - Fix import order in SortByPaidCommandTest - Remove duplicate SortPaidCommandTest file
- Add sortbypaid command documentation to User Guide - Add sortbypaid to Command Summary table - Add sortbypaid command reference in help/CommandReference.md - Include usage examples and guidance for trainers - Document sorting behavior: unpaid first, paid second
Update UML diagram
Add personDeadlineComparator, sortByDeadlineCommand, SortByDeadlineCommandParser.
Add Dashboard panel and organise UI components
Updated the tests pertaining to said class.
- Create SortByPaidCommandDiagram.puml: Detailed class diagram with proper UML notation - Shows fields and methods for all classes (SortByPaidCommand, Person, Paid, etc.) - Displays command hierarchy and model relationships - Includes comprehensive notes explaining functionality - Create SortByPaidSequenceDiagram.puml: Complete execution flow sequence - Shows user input to result flow - Displays all method calls and delegation - Covers error handling and success paths - Create SortByPaidActivityDiagram.puml: Sorting workflow activity - Shows decision points and validation - Displays sorting logic and result display - Update DeveloperGuide.md with sortbypaid implementation section - Document all components and relationships - Reference all three diagram types - Explain sorting behavior and integration
- Remove trailing whitespace from SortByPaidActivityDiagram.puml - Remove trailing whitespace from CommandReference.md - Fix CI Checkstyle violations for PlantUML files - Ensure all files pass CI checks
Add sortbypaid command functionality
Update documentation sortbypaid
Add uml diagrams sortbypaid
All checkstyle tests now pass.
Fixed trailing whitespace error
Fixed whitespace error again
Missing @OverRide for sortpersonlistbypaid
Add import to sortbypaidtest.
Add SortByDeadline functionality
Update help and session in UserGuide.md
Functional Code Bug Fix
Changed from addressbook.json.
Edit FitBook.json
Updated UG to explicitly state that gender values accept mixed-case inputs (e.g., Male, FEMALE, Non-Binary) to match current implementation and resolve documentation bug.
Clarify gender values are case-insensitive in the user gide
UG updates
Find Command fix
fix issues
- Remove placeholder content from Developer Guide - Replace AddressBook references with FitBook where appropriate - Add UC08 and UC09 use cases for undo/redo feature - Update undo/redo examples to use FitBook-specific commands - Fix UML sequence diagram: CommandResult creation notation - Fix UML sequence diagram: SortByPaidCommand creation notation - Change actor from User to Trainer for consistency - Fix Checkstyle warnings in DeadlineCommand - Enhance site navigation with detailed subsections
Fix PE-D documentation bugs and UML diagram issues
Updated these sections in UG.
faq and known issues in UG
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.