[CS2103T-T03-1] MediConnect #68
Open
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
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #68 +/- ##
=============================================
- Coverage 72.15% 55.72% -16.44%
- Complexity 399 560 +161
=============================================
Files 70 102 +32
Lines 1232 2281 +1049
Branches 125 283 +158
=============================================
+ Hits 889 1271 +382
- Misses 311 915 +604
- Partials 32 95 +63
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
jedidiahC
pushed a commit
to jedidiahC/tp
that referenced
this pull request
Mar 15, 2023
Added navigation of user to user guide
weekiat-douze
pushed a commit
to weekiat-douze/tp
that referenced
this pull request
Mar 25, 2023
V1.3 [Add tag colours to labels]
UG Remove repeated line
Fix display command bug
Disable person view display when no contacts are available
The message displayed when the patient added by the user already exists in the list sometimes includes the wrong role for that person. Let's update the error message.
The message displayed when the doctor added by the user already exists in the list sometimes includes the wrong role for that person. Let's update the error message.
* 'master' of https://github.com/thennant/tp: Disable person view display when no contacts are available Fix display bug when deleting appointment Remove repeated line Update ug Update ug Rename files Fix checkstyle Update sample data tags Adjust layout Make empty cells transprarent Update prescription box Hide details on the person list panel Update UG Update UG Update the developer guide to fit the features for v1.3 Change user stories to fit hospital administrator Update User Guide Update documentation for list commands Add listDoctors and listPatients commands
…ds, the edit command gives an inaccurate error message: "The person you are looking for cannot be found, below is a list of people with similar names". This is confusing to the user as the list of names given are not similar names Let's change the error message to say: "The person you are looking for cannot be found"
AssertCommandFailure was causing the test to fail Let's update the tests
The user might have made a mistake typing the wrong nric number and wishes to edit the nric Let's update the editcommand to allow the editing of NRIC field
…letes an appointment of a patient that is not in the current list (issue 119) This can be confusing for the user when the user enters the command. Let's add the error message to notify the user that the patient is not in the list
Command Summary does not show tell the user any information about the command other than its purpose More information to make it more useful as a summary Let's add the syntax and an example
Reorder arguments in unprescribe to make it clearer
Current message is ambiguous about cost being 0. Could be clearer to users. Let's update UG and Cost Constraint Message to be clearer.
Currently UG does not reflect the situation where a duplicate medication is prescribed It is better to inform our users. Let's update the UG with this information
…s not" This reverts commit 0c7e63e.
Fix double punctuation in success message
The User Guide has some minor spelling mistakes and vague restrictions for the "Address" input values. Let's add restrictions for "Address" and correct the spelling mistakes.
Bugfix issue89
Bugfix issue113
Bugfix issue119
Fix message displayed when failing to add a Patient
fix DG page breaks
# Conflicts: # docs/UserGuide.md
User Guide: Update Images to be centered
fix page break for DG
User Guide: Fix formatting errors
fix page break for DG
* 'master' of https://github.com/thennant/tp: fix page break for DG User Guide: Fix formatting errors fix page break for DG fix DG page breaks User Guide: Update Images to be centered Update formatting finalised layout of UG added more pg breaks testing line breaks in UG
Update user guidev5
fix UG formatting
change bill image size
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.
MediConnect helps hospital admin staff manage the details of doctors and patients such as appointments, medication, and bills.
It is optimized for CLI users so that frequent tasks can be done faster by typing in commands.