-
Notifications
You must be signed in to change notification settings - Fork 712
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
[CS2103T-T08-3] MedDict #56
Open
chengzihao123
wants to merge
511
commits into
nus-cs2103-AY2425S1:master
Choose a base branch
from
AY2425S1-CS2103T-T08-3: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
[CS2103T-T08-3] MedDict #56
chengzihao123
wants to merge
511
commits into
nus-cs2103-AY2425S1:master
from
AY2425S1-CS2103T-T08-3: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
Judy1x4
pushed a commit
to Judy1x4/tp
that referenced
this pull request
Oct 16, 2024
…add-feature Add Comment prefix in CLI Syntax to use in AddCommand
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
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.
MedDict Provide a more specialised one-stop-for-all medical related information to physiotherapists' patients, schedules, appointments and to keep track of patients' medical information, progression, medical history etc. Optimized for physiotherapists who may have to keep close track of all their patients' detailed progress, whilst having to juggle with having many patients.