Resolve conflict locally#6
Open
sandyk0105 wants to merge 294 commits intosandyk0105:108-add-appointment-markascompletefrom
Open
Resolve conflict locally#6sandyk0105 wants to merge 294 commits intosandyk0105:108-add-appointment-markascompletefrom
sandyk0105 wants to merge 294 commits intosandyk0105:108-add-appointment-markascompletefrom
Conversation
The current implementation has less information about our main feature add createDoctorCommand and createPatientCommand implementation detail in developer guide
…am for CreatePatientCommand Commit Body: The Developer Guide and sequence diagram currently use inaccurate terminology, referring to "MedDict database in address book" instead of "addressbook". -Updated terminology in Developer Guide to align with MedDict's purpose as a patient management application. -Adjusted the execute() method call to pass the Model.
UML Modification
-Adding three unit test for the MarkAppointmentCommand to verify its behavior when marking an appointment. -These test ensures that the appointment status is correctly updated to reflect that it has been marked, helping to validate that the command functions as intended. This change: Adds three dedicated test to enhance coverage for appointment management. Aims to prevent future regressions by confirming the correct appointment status updates when marking.
Too many characters in one line Break the particular line of statement into two lines
MarkAppointmentCommandTest: add unit test for marking appointments
Add extra Logging
Add three unit test for the RemarkCommand to verify its behavior when adding or updating remarks for an appointment. This test ensures that remarks are correctly applied to the relevant appointment, validating the command’s functionality and supporting consistent behavior. This change introduces a focused test to improve coverage for the remark functionality and aims to prevent future regressions by ensuring remarks update as expected.
Unused import statement found Remove unused import statement
Extra separation Remove extra space
Unused import statement found Remove unused statement
…rampuml Update DeleteSequenceDiagram
Changes some words Adds createPatient feature
Update UserGuide.md
RemarkCommandTest: add unit test for remark functionality
The MedDict user guide lacks specific details and examples for some features, which can make it harder for users to fully understand the commands. - Add detailed descriptions of input formats and constraints - Include examples to demonstrate expected outputs and error messages - Refine language for consistent terminology and readability
The current version has no new line for every example Add new line to ensure consistency
Removw white spaces
…yxiangs/tp into branch-UserGuide-UpdateFeature
Branch user guide update feature
Modify existing commands
Some of the command get modify to be less tedious Update all the command accordingly
Update user guide
More changes to DG
More changes to DG
More changes to DG
More changes to DG
Fix UG inconsistency
Change PATIENT_ID to ID for delete command
Edit UserGuide
More changes to DG
More changes to DG
More changes to DG
More changes to DG
Add known issue
More changes to DG
Fix known issue
Enhance credit
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.