Skip to content
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

V1.2 #80

Open
wants to merge 299 commits into
base: master
Choose a base branch
from
Open

V1.2 #80

wants to merge 299 commits into from

Conversation

chokxy
Copy link

@chokxy chokxy commented Oct 17, 2018

No description provided.

brandonccm1996 and others added 30 commits September 20, 2018 00:01
Added AddAppt command for patient timetable
Command does nothing now.
Takes in index integer and medhistory string.
Previous 2 commits were changes/updates to Logic.
This commit changes medhistory string to be a MedHistory class instead.
Followed tutorial answers. From observation, a $medhistory entry is seen in patient card in the GUI.
Add and Edit Command for MedHistory does not work here yet.
XML == Extensible Markup Language. Further read up required on the importance of this class.
Observation: GUI no longer shows EMPTY_MEDHISTORY which was $medhistory. Now it shows nothing.
This is the end of the tutorial to add a new command. In this commit, the error message got replaced with real logic to edit the medical history of a patient.
Found in AddressBookParser and AddressBookParserTest.
Add a Codacy badge to README.adoc
-Fixed codacy badge
-Removed unused CI services
- Add Appt to model component
- Add test for Appt (to test the equals method).
- Modify AddApptCommand to now take in an Appt instead of a String.
brandonccm1996 and others added 29 commits October 17, 2018 21:11
UGDG updates for medical history.
Updated User Guide and Developer Guide for addinfo
V1.2 Changing MedHistory to use Set
Multiple Arguments for Additional Info and Date of Birth Implementation
Find by keyword with Prefix V1.2
…h_to_merge_from(never_add_hovan's_pr)' into v1.2
# Conflicts:
#	src/main/java/seedu/address/logic/parser/AddCommandParser.java
#	src/main/java/seedu/address/logic/parser/AddMedicalReportCommandParser.java
#	src/main/java/seedu/address/model/medicalreport/MedicalReport.java
#	src/main/java/seedu/address/model/person/Person.java
#	src/main/java/seedu/address/storage/XmlAdaptedPerson.java
#	src/test/java/seedu/address/testutil/ReportBuilder.java
#	src/test/java/seedu/address/testutil/TypicalPersons.java
Merge with xhxh96 v1.2
auskure pushed a commit to auskure/main that referenced this pull request Nov 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants