-
Notifications
You must be signed in to change notification settings - Fork 521
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
[CS2103-T17-2] ContactSwift #177
Open
shayaansultan
wants to merge
380
commits into
nus-cs2103-AY2324S2:master
Choose a base branch
from
AY2324S2-CS2103T-T17-2:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
[CS2103-T17-2] ContactSwift #177
shayaansultan
wants to merge
380
commits into
nus-cs2103-AY2324S2:master
from
AY2324S2-CS2103T-T17-2:master
Conversation
This file contains 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
Previously there were a few lines that were partially covered by tests. This commit fixes that by adding tests for those lines.
Fix bug regarding name-based deletion
Updated the Developer Guide to include comprehensive documentation for the Filter and ShowAll commands. This enhancement outlines the implementation details, rationale, and design considerations for these features. - The Filter Command section explains its parsing and execution process, highlighting the use of predicates for dynamic employee filtering based on specified criteria like name, role, team, and tags. - The ShowAll Command section describes its functionality to reset and display the complete list of employees, detailing its integration with the model to clear any active filters. This documentation provides clarity on the features' workings, assisting developers in understanding and maintaining the codebase. The choice of implementation for each command is justified with a focus on flexibility, user experience, and adherence to the application's architectural standards. The detailed documentation ensures that future enhancements or modifications to these features can be made efficiently and with a thorough understanding of their initial design and purpose.
Updated the Developer Guide to include comprehensive documentation for the Filter and ShowAll commands. This enhancement outlines the implementation details, rationale, and design considerations for these features. - The Filter Command section explains its parsing and execution process, highlighting the use of predicates for dynamic employee filtering based on specified criteria like name, role, team, and tags. - The ShowAll Command section describes its functionality to reset and display the complete list of employees, detailing its integration with the model to clear any active filters. This documentation provides clarity on the features' workings, assisting developers in understanding and maintaining the codebase. The choice of implementation for each command is justified with a focus on flexibility, user experience, and adherence to the application's architectural standards. The detailed documentation ensures that future enhancements or modifications to these features can be made efficiently and with a thorough understanding of their initial design and purpose.
Update developer guide
…to 'Employee' in UML diagrams
The Developer Guide has been updated to include details about the implementation of the delete command. This includes the handling of deletion by name in case of duplicate names and deletion by unique id. This information provides clarity on how these features are implemented and can assist other developers in understanding the codebase.
Update Developer Guide with delete command details
Update Developer Guide with UID info
update DG to include AddCommand implementation
This commit introduces a new Description class in the task package. The Description class represents a Task's description in the address book. It includes validation checks to ensure that the description is not blank. The class includes methods for constructing a Description object, validating a description string, and overriding the toString, equals, and hashCode methods. This new class provides a structured way to handle task descriptions, improving code readability and maintainability.
This commit introduces a new Task class in the task package. The Task class represents a task with a Description and a boolean status indicating if the task is done. The class includes methods for constructing a Task object, marking and unmarking a task as done, and overriding the toString, equals, and hashCode methods. This new class provides a structured way to handle tasks, improving code readability and maintainability.
This commit introduces a new TodoList class in the todoList package. The TodoList class represents a list of tasks for each employee. The class includes methods for adding, removing, marking, and unmarking tasks, and overriding the toString method. This new class provides a structured way to manage tasks, improving code readability and maintainability.
This commit adds unit tests for the Description, Task, and TodoList classes in the task and todoList packages. The tests cover the main functionalities of these classes, including validating descriptions, checking task equality, and managing tasks in a todo list. These tests help ensure the correctness of the code and facilitate future refactoring and enhancement.
The application currently lacks a way to add tasks to an employee's todo list. This is a crucial feature for task management in a team. To address this, the AddTaskCommand class was added. This class extends the Command class and implements the functionality to add a task to an employee's todo list. The command takes a unique employee id and a task description as input, and adds a new task with the given description to the todo list of the corresponding employee. This addition improves the application's task management capabilities and allows users to assign tasks to employees.
The Description class previously allowed any string to be used as a task description. This could lead to undesired behavior, such as descriptions with special characters or blank descriptions. To improve data integrity, the Description class has been modified to only allow alphanumeric characters and spaces in descriptions. In addition, descriptions cannot be blank, null, or consist only of whitespace. This change ensures that all task descriptions adhere to the defined format and improves the robustness of the application.
The application currently lacks a way to parse user input for the addTask command. This is a crucial feature for task management in a team. To address this, the AddTaskCommandParser class was added. This class implements the Parser interface and provides a parse method that creates an instance of the AddTaskCommand class from a given user input string. In addition, the AddressBookParser class was updated to handle the addTask command. The parseCommand method of this class was modified to recognize the addTask command word and call the AddTaskCommandParser. These changes improve the application's task management capabilities and allow users to assign tasks to employees using the addTask command.
Improve UI aesthetics
Fix Formatting
Add data archive section, update ug, fix hyperlinks
Fix typos in DG
169-uml-modifications
update the default employees
Update ug typo
Add DG Appendix: Effort
139-dg-planned-enhancements
Finalize UG
Fix typos in UG
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
ContactSwift streamlines contact management with its intuitive interface, offering quick access to contact details and seamless integration with productivity tools, ideal for busy professionals seeking efficiency in their daily tasks.