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

[CS2113-T10-2] BookBob #11

Open
wants to merge 871 commits into
base: master
Choose a base branch
from
Open
Changes from 1 commit
Commits
Show all changes
871 commits
Select commit Hold shift + click to select a range
2ea68dd
add author
Nov 6, 2024
cc7f9ff
Merge pull request #287 from kaboomzxc/master
kaboomzxc Nov 6, 2024
0f09af9
fix author
Nov 7, 2024
e1539c7
Merge pull request #288 from kaboomzxc/master
kaboomzxc Nov 7, 2024
2ab0787
Change the help output to guide the users correctly
yentheng0110 Nov 7, 2024
4fe89bc
Make the error message printed out for find command consistent for a…
yentheng0110 Nov 7, 2024
bbb00d1
Merge pull request #290 from yentheng0110/YenTheng-FinalCheckBeforeRe…
kaboomzxc Nov 7, 2024
e92f54c
Fix bugs in the commands
Nov 7, 2024
21e30e3
Fix typo issue
G13nd0n Nov 7, 2024
135a41c
Fixed typo
G13nd0n Nov 7, 2024
5defa7b
Merge pull request #292 from G13nd0n/master
yentheng0110 Nov 7, 2024
b8129ad
Fix minor typo in Main.java
yentheng0110 Nov 7, 2024
d061cfb
Merge pull request #293 from yentheng0110/master
kaboomzxc Nov 7, 2024
c776614
Update the DeveloperGuide based on feedback received
Nov 8, 2024
572d071
Extract certain variables used throughout the tests
Nov 8, 2024
222578f
Merge branch 'master' of https://github.com/AY2425S1-CS2113-T10-2/tp
coraleaf0602 Nov 8, 2024
25e2a43
Add interface for file operations in FileHandler
coraleaf0602 Nov 8, 2024
92bdc8c
Add interface for converting file output
coraleaf0602 Nov 8, 2024
8c9d3b0
Modify Patient class to include OutputConversion interface
coraleaf0602 Nov 8, 2024
a7202fe
Modify Patient class to include OutputConversion interface
coraleaf0602 Nov 8, 2024
c9357cb
Modify AppointmentRecord class to include FileOperation interface
coraleaf0602 Nov 8, 2024
d4f007d
Modify Record class to include FileOperation interface
coraleaf0602 Nov 8, 2024
8fca0b2
Remove unused imports and refactored code
coraleaf0602 Nov 8, 2024
c9acf3c
Modify the class files to extract duplicate methods, shift delete and…
Nov 9, 2024
1fd1b22
Modify attribute of Appointment
Nov 9, 2024
24e4948
Merge pull request #343 from G13nd0n/master
yentheng0110 Nov 9, 2024
f4880d4
Import FileHandler class to Records class to use static methods
coraleaf0602 Nov 9, 2024
aab469a
Update authorship for methods
coraleaf0602 Nov 9, 2024
1802255
Delete commented code from FileHandler
coraleaf0602 Nov 9, 2024
3835034
Merge branch 'master' into Inheritance-File-Handling
coraleaf0602 Nov 9, 2024
12a6c18
Change test method to adhere to code structure in FileHandler
coraleaf0602 Nov 9, 2024
ed519b9
Merge branch 'Inheritance-File-Handling' of https://github.com/corale…
coraleaf0602 Nov 9, 2024
fe2608f
Bug Fix: Missing semicolon
coraleaf0602 Nov 9, 2024
b58a45b
Merge pull request #342 from coraleaf0602/Inheritance-File-Handling
yentheng0110 Nov 9, 2024
7d9413b
before_pull
PrinceCatt Nov 9, 2024
3982195
Merge branch 'master' of https://github.com/PrinceCatt/tp
PrinceCatt Nov 9, 2024
867349f
Merge pull request #344 from PrinceCatt/master
yentheng0110 Nov 9, 2024
8c58865
Merge branch 'master' of https://github.com/AY2425S1-CS2113-T10-2/tp
Nov 9, 2024
35d8081
Fix bugs in the add function
Nov 9, 2024
999952d
Fix minor bug in the removePastAppointment Function
Nov 9, 2024
73994fd
Convert DOB to localdate and change corresponding methods
PrinceCatt Nov 9, 2024
2b86262
Merge pull request #345 from G13nd0n/master
yentheng0110 Nov 9, 2024
f45869a
fix_CI
PrinceCatt Nov 9, 2024
b5c1483
Merge branch 'master' into master
PrinceCatt Nov 9, 2024
84f1734
fix_CI2
PrinceCatt Nov 9, 2024
7d64843
Merge branch 'master' of https://github.com/PrinceCatt/tp
PrinceCatt Nov 9, 2024
c327a19
fix_CI3
PrinceCatt Nov 9, 2024
a9040f8
minor_change
PrinceCatt Nov 9, 2024
903c37c
Update CommandHandler.java
PrinceCatt Nov 9, 2024
e7124bc
Update CommandHandler.java
PrinceCatt Nov 9, 2024
9c69fb2
Update CommandHandler.java
PrinceCatt Nov 9, 2024
a280fa0
Update CommandHandler.java
PrinceCatt Nov 9, 2024
86828da
Update CommandHandler.java
PrinceCatt Nov 9, 2024
19b22de
Update CommandHandler.java
PrinceCatt Nov 9, 2024
cf6962b
Update CommandHandler.java
PrinceCatt Nov 9, 2024
ef3a073
Update CommandHandler.java
PrinceCatt Nov 9, 2024
5648182
Update CommandHandler.java
PrinceCatt Nov 9, 2024
91967db
Update CommandHandler.java
PrinceCatt Nov 9, 2024
1328918
Update CommandHandler.java
PrinceCatt Nov 9, 2024
498e7be
fix_CI
PrinceCatt Nov 9, 2024
78b9051
merge
PrinceCatt Nov 9, 2024
d448f1a
Add_authorship
PrinceCatt Nov 9, 2024
54e61ca
add space between if and (
PrinceCatt Nov 9, 2024
e38d3c0
Merge pull request #346 from PrinceCatt/master
yentheng0110 Nov 9, 2024
8859e99
Create a Parser class for more OOP
yentheng0110 Nov 9, 2024
49eb9f3
Merge pull request #348 from yentheng0110/YenTheng-CreateParserClass
PrinceCatt Nov 9, 2024
063452c
ban | in input
PrinceCatt Nov 9, 2024
1236aa3
Edit UG for add command such that DOB inputted is of the correct DD-M…
yentheng0110 Nov 9, 2024
30e0c33
Fix the bug so that when multiple mandatory fields are not provided, …
yentheng0110 Nov 9, 2024
bd694ee
Fix the bug that name input is not given as the first input but name …
yentheng0110 Nov 9, 2024
3b57c5e
Fix the bug for edit command, when the user forgets to provide update…
yentheng0110 Nov 9, 2024
ad65b8a
Fix the bug that home addresses with space and hyphens are accepted a…
yentheng0110 Nov 9, 2024
c6886a5
Make sure phone number is not a mandatory field, a user won't be prom…
yentheng0110 Nov 9, 2024
005ade2
Make sure that sex is not a mandatory field, a user is not prompted t…
yentheng0110 Nov 9, 2024
8f834ae
Edit CommandHandler.java, adding assertions, fixing logical bugs and …
yentheng0110 Nov 9, 2024
c365790
Update the Parser.java to delegate the validation of parameters to Co…
yentheng0110 Nov 9, 2024
9f84e79
Commit the log file
yentheng0110 Nov 9, 2024
9e721f2
Delete MainTest.java as JunZheng will be implementing a different one
yentheng0110 Nov 9, 2024
b412e6e
Fix bug to accept spaces, hyphens, commas and slashes for names to be…
yentheng0110 Nov 9, 2024
7b03677
Modify test methods in CommandHandler.java to pass all tests
yentheng0110 Nov 9, 2024
0443298
Fix checkstyleMain issue
yentheng0110 Nov 9, 2024
7dd8d19
Close issue #300 raised by peer tester
yentheng0110 Nov 9, 2024
9dddc78
Merge pull request #349 from PrinceCatt/master
yentheng0110 Nov 9, 2024
ef62847
Merge pull request #356 from yentheng0110/master
coraleaf0602 Nov 10, 2024
c906e9f
Add formatting for phone number
coraleaf0602 Nov 10, 2024
e67250f
Merge pull request #357 from coraleaf0602/Format-Phone-Number
kaboomzxc Nov 10, 2024
0220c0e
Main Sequence Diagram
Nov 10, 2024
41e0838
Object Diagram on initialisation of BookBob
Nov 10, 2024
7c93730
Update Main Class Diagram
Nov 10, 2024
e6d0047
Update CommandHandler Class Diagram
Nov 10, 2024
9ccd465
Updated FileHandler
Nov 10, 2024
478db26
Update Records Class Diagram
Nov 10, 2024
01bca21
update AppointmentRecord Class Diagram
Nov 10, 2024
9bde22f
update Appointment Class Diagram
Nov 10, 2024
eec64d9
update Visit Class Diagram
Nov 10, 2024
2f9f6f7
update Patient Class Diagram
Nov 10, 2024
44b088a
update ObjectDiagramBeforeAddVisit object diagram
Nov 10, 2024
d78df8f
Update Object Diagram After Add Visit
Nov 10, 2024
e16007b
updated AddVisitSequenceDiagram
Nov 10, 2024
c4e5d7c
update ObjectDiagramBeforeAddPatient Object Diagram
Nov 10, 2024
23eb86b
Update ObjectDiagramAfterAddPatient object diagram
Nov 10, 2024
d53074f
Update New Patient Sequence Diagram
Nov 10, 2024
9c818e7
Fix bugs based on feedbacks given and update UG
Nov 10, 2024
c502adb
Update Object Diagram for Before Appointment and After Appointment ,
Nov 10, 2024
8b49be6
Fix CI issue
Nov 10, 2024
de3b382
Fix CI and checkstyle issue
Nov 10, 2024
fc7a85d
Update both DG and UG for the amendments made
Nov 10, 2024
8da9de3
update UG
Nov 10, 2024
a768257
Update findAppointment to be case insensitive
Nov 10, 2024
729c308
Merge pull request #358 from G13nd0n/master
kaboomzxc Nov 10, 2024
210e62c
Merge pull request #359 from kaboomzxc/Class&Object&Seq_Diagrams_10No…
G13nd0n Nov 10, 2024
30dc683
Merge pull request #360 from kaboomzxc/update_UG_10Nov_(1)
G13nd0n Nov 10, 2024
aeb82ec
Update UG
Nov 10, 2024
96b84cd
update UG
Nov 10, 2024
b66d567
update UG typo
Nov 10, 2024
57b4414
Make find visit by diagnosis and by medication case insensitive
PrinceCatt Nov 10, 2024
a3c9efd
update UG
Nov 10, 2024
1d14d97
Merge pull request #362 from kaboomzxc/update_UG_10Nov_(2)
kaboomzxc Nov 10, 2024
1d2ed0b
Merge pull request #363 from PrinceCatt/master
kaboomzxc Nov 10, 2024
c0265e2
update UG
Nov 10, 2024
db66090
Merge pull request #364 from kaboomzxc/master
kaboomzxc Nov 10, 2024
6cbfe99
Add error report and auto-exiting when visit dates and DOB corrupted
PrinceCatt Nov 10, 2024
db366fe
Include phone number and NRIC checks
PrinceCatt Nov 10, 2024
4568491
Include appointment record checks
PrinceCatt Nov 10, 2024
a06a297
fix_CI
PrinceCatt Nov 10, 2024
34b62b9
Merge pull request #365 from PrinceCatt/master
kaboomzxc Nov 10, 2024
2d58970
Fix bug of adding duplicate patient records
Nov 10, 2024
e4680ad
Modify the deleteAppointment to allow the deletion of appointment reg…
Nov 10, 2024
541e5e3
Fix bug of list command
Nov 10, 2024
ce8c9c8
Merge pull request #366 from G13nd0n/master
yentheng0110 Nov 10, 2024
7aad201
update UG
Nov 10, 2024
8645874
update ug
Nov 10, 2024
d730145
Merge pull request #367 from kaboomzxc/update_UG_11Nov_(1)
kaboomzxc Nov 10, 2024
5438ac4
update UG
Nov 10, 2024
0253cf2
Merge pull request #368 from kaboomzxc/update_UG_11Nov_(1)
kaboomzxc Nov 10, 2024
7170a3e
successfully solved :
Nov 10, 2024
5b23114
fix CI
Nov 10, 2024
f8ea2d9
fix ci
Nov 10, 2024
56deac2
Merge pull request #369 from kaboomzxc/addVisit_NoDuplicate_Date&Time…
yentheng0110 Nov 11, 2024
b9b70cf
Modify Records.java to address many bug reports about adding patients…
yentheng0110 Nov 11, 2024
eb014f2
Add author tag
yentheng0110 Nov 11, 2024
a07e716
Merge pull request #370 from yentheng0110/YenTheng-BanTheAdditionOfDu…
kaboomzxc Nov 11, 2024
92f9622
Modify DG to guide the users correctly
yentheng0110 Nov 11, 2024
5b2e58b
Modify findVisit to include patient's name when printing out (besides…
yentheng0110 Nov 11, 2024
cbaa0fb
Extract the method as printUnknownCommand(String command)
yentheng0110 Nov 11, 2024
581d2af
Remove log file
yentheng0110 Nov 11, 2024
46e2b04
Fix a bug in checking for phone number corruption
PrinceCatt Nov 11, 2024
829da86
Merge pull request #372 from PrinceCatt/master
kaboomzxc Nov 11, 2024
4d03580
Modify UG to guide users correctly
yentheng0110 Nov 11, 2024
1501689
Add an extra space between } and catch for better readability
yentheng0110 Nov 11, 2024
b067528
Merge branch 'master' into YenTheng-AddressBugReports
yentheng0110 Nov 11, 2024
d40c657
Bold add and addVisit command so that users can differentiate between…
yentheng0110 Nov 11, 2024
44e8155
Merge branch 'YenTheng-AddressBugReports' of https://github.com/yenth…
yentheng0110 Nov 11, 2024
aa23e9f
Modify UG for numbering issues and minor formatting issues
yentheng0110 Nov 11, 2024
2a26cd2
Merge pull request #371 from yentheng0110/YenTheng-AddressBugReports
kaboomzxc Nov 11, 2024
d8b98ac
update UG
Nov 11, 2024
c24de97
Merge pull request #373 from kaboomzxc/11Nov330pm
kaboomzxc Nov 11, 2024
8461e4d
51 tests case passed,
Nov 11, 2024
af9c8d2
removed unused imports and add author
Nov 11, 2024
752356b
fix CI
Nov 11, 2024
22e2389
update name
Nov 11, 2024
a37ba9a
changed correct name
Nov 11, 2024
d28091a
Merge pull request #375 from kaboomzxc/ParserTest_11NovEvening_(2)
yentheng0110 Nov 11, 2024
7cce9e1
"addVisit" command prevents addition of future dates.
Nov 11, 2024
ce55186
fix ci
Nov 11, 2024
20ff5f5
fix ci
Nov 11, 2024
95c7f24
fix ci indentation
Nov 11, 2024
b3ce904
Merge pull request #376 from kaboomzxc/addVisit_NoFutureDate
coraleaf0602 Nov 11, 2024
659fa2c
Update UG and DG based on the refinement of methods
Nov 11, 2024
68660b5
"add" command prevents addition of future dates.
Nov 11, 2024
dd1a3d9
Fix issue of diagram for DG
Nov 11, 2024
7a336b2
fix CI
Nov 11, 2024
b783938
fix ci
Nov 11, 2024
b1557fd
Merge pull request #377 from G13nd0n/master
kaboomzxc Nov 11, 2024
5f0dead
Merge pull request #379 from kaboomzxc/add_Command_NoFutureDate
kaboomzxc Nov 11, 2024
beae49b
make diagrams bigger
Nov 11, 2024
5a29961
Merge pull request #380 from kaboomzxc/tryMakeDiagramsBigger
kaboomzxc Nov 11, 2024
fb253d6
Change DG formatting issues for better readability
yentheng0110 Nov 11, 2024
55ad2ab
Modify the command summary reference as there are some incorrect guid…
yentheng0110 Nov 11, 2024
fbc64bb
Merge pull request #382 from yentheng0110/YenTheng-AddressBugReports
kaboomzxc Nov 11, 2024
1d0e803
Modify Table of Contents to make sure all links are clickable and Com…
yentheng0110 Nov 11, 2024
436887d
Merge pull request #383 from yentheng0110/YenTheng-AddressBugReports
kaboomzxc Nov 11, 2024
4f63a12
Fix formatting issues in DG
yentheng0110 Nov 11, 2024
43f0eea
try fix broken section formatting
Nov 11, 2024
7c7ee45
Merge pull request #384 from kaboomzxc/master
kaboomzxc Nov 11, 2024
5f6dd86
fix DG formatting
Nov 11, 2024
9600f6a
Merge pull request #386 from kaboomzxc/master
kaboomzxc Nov 11, 2024
9b3faea
update UG, DG
Nov 11, 2024
1b07ef5
Merge pull request #387 from kaboomzxc/fixUG&DG_12Nov130am
kaboomzxc Nov 11, 2024
af9dd15
fix "find" finding DOB,
Nov 11, 2024
47f3429
fix CI
Nov 11, 2024
c90da96
fix ci
Nov 11, 2024
83f9467
Update PPP
yentheng0110 Nov 11, 2024
a2c4618
Merge branch 'master' into YenTheng-AddressBugReports
yentheng0110 Nov 11, 2024
33fd955
Merge pull request #390 from yentheng0110/YenTheng-AddressBugReports
yentheng0110 Nov 11, 2024
9a94998
fix ci
Nov 11, 2024
64fa592
Fix formatting issues for PPP
yentheng0110 Nov 11, 2024
e95670a
Merge pull request #391 from yentheng0110/master
yentheng0110 Nov 11, 2024
6dddab0
fix ci
Nov 11, 2024
d95ab2d
Fix issues in formatting for PPP
yentheng0110 Nov 11, 2024
eab0b31
fix ci
Nov 11, 2024
42e0bc7
Merge pull request #392 from yentheng0110/master
yentheng0110 Nov 11, 2024
a4ea027
fix ci
Nov 11, 2024
bf37f5f
fix ci
Nov 11, 2024
7a0396a
fix ci
Nov 11, 2024
ad3ab32
Merge pull request #388 from kaboomzxc/fix_find_dob
kaboomzxc Nov 11, 2024
bb5f6af
Shorten summary of contributions
yentheng0110 Nov 11, 2024
9a2195d
Fix minor formatting issues in PPP
yentheng0110 Nov 11, 2024
fc20cd7
Merge pull request #393 from yentheng0110/master
yentheng0110 Nov 11, 2024
be32a15
update class diagrams Record and AppointmentRecord
Nov 11, 2024
50acd59
Final check for PPP
yentheng0110 Nov 11, 2024
a56c6e2
Merge pull request #394 from kaboomzxc/master
kaboomzxc Nov 11, 2024
435eaa7
Merge pull request #395 from yentheng0110/YenTheng-FinalCheckForUG
yentheng0110 Nov 11, 2024
52cc7a5
Fix help command and command summary in UG to guide the users correctly
yentheng0110 Nov 11, 2024
0fb6a44
Shorten Overview of PPP
yentheng0110 Nov 11, 2024
e1997b4
Merge pull request #396 from yentheng0110/YenTheng-FinalCheckForUG
yentheng0110 Nov 11, 2024
73ea397
JunZheng PPP
Nov 11, 2024
5f098ac
Merge pull request #398 from kaboomzxc/JunZheng_PPP_(1)
kaboomzxc Nov 11, 2024
e9763fd
update JZ PPP
Nov 11, 2024
5b0cdcb
Merge pull request #399 from kaboomzxc/master
kaboomzxc Nov 11, 2024
46076eb
fix typo JZ PPP
Nov 12, 2024
25c27ed
update Main Object Diagram
Nov 12, 2024
c3930b5
Merge pull request #400 from kaboomzxc/master
kaboomzxc Nov 12, 2024
7f34f9b
"patient record management" header in DG reduce size
Nov 12, 2024
f251b22
Merge pull request #401 from kaboomzxc/master
kaboomzxc Nov 12, 2024
2f5c16a
Modify UG for add command to include input validation section, explan…
yentheng0110 Nov 12, 2024
90127d3
Modify extractGender(String input) in CommandHandler.java such that o…
yentheng0110 Nov 12, 2024
cfc4c90
Merge pull request #402 from yentheng0110/YenTheng-FinalCheckBeforeRe…
yentheng0110 Nov 12, 2024
ce0ff8d
Fix minor formatting issues in UG
yentheng0110 Nov 12, 2024
7bd9b28
Merge pull request #403 from yentheng0110/master
yentheng0110 Nov 12, 2024
663ad03
Update PPP
Nov 12, 2024
66b4b4f
update JZ PPP
Nov 12, 2024
c9315d4
Merge pull request #406 from kaboomzxc/master
kaboomzxc Nov 12, 2024
3af81e1
update JZ PPP
Nov 12, 2024
3cc16f4
Merge pull request #407 from kaboomzxc/master
kaboomzxc Nov 12, 2024
ee6b995
Update the enhancement contributed
Nov 12, 2024
804f5e7
Merge pull request #405 from G13nd0n/master
G13nd0n Nov 12, 2024
644b435
update JZ PPP
Nov 12, 2024
e02a0fc
Merge pull request #408 from kaboomzxc/master
kaboomzxc Nov 12, 2024
ac258cf
Update PPP
coraleaf0602 Nov 12, 2024
995812b
Update PPP
coraleaf0602 Nov 12, 2024
03c92c3
Merge pull request #409 from coraleaf0602/master
coraleaf0602 Nov 12, 2024
ba7678a
Update coraleaf0602.md
coraleaf0602 Nov 12, 2024
7629020
fix Typos in UG
Nov 27, 2024
7e9b97f
fix CI
Nov 27, 2024
b5a046f
fix CI
Nov 27, 2024
b66b9a3
fix CI
Nov 27, 2024
3046b63
Merge pull request #410 from kaboomzxc/master
kaboomzxc Nov 27, 2024
78dfe4e
update JZ PPP
Nov 27, 2024
7426ee1
Merge pull request #411 from kaboomzxc/master
kaboomzxc Nov 27, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update PPP
g13nd0n-admin committed Nov 12, 2024
commit 663ad0312c956a214c9eff6b4358696ecb37935a
24 changes: 17 additions & 7 deletions docs/team/g13nd0n.md
Original file line number Diff line number Diff line change
@@ -1,11 +1,12 @@
# Glendon Tan - Project Portfolio Page

## Overview
BookBob is a desktop application tailored for Dr Bob’s private General Practitioner clinic. It facilitates the storage
BookBob is a desktop application tailored for Doctor's private General Practitioner clinic. It facilitates the storage
and retrieval of patient information, including names, NRICs, genders, dates of birth, phone numbers, home addresses,
allergies, medical histories and visit records with details on diagnoses and prescribed medications. BookBob also helps
Dr Bob stay organised by tracking daily appointments and providing reminders each morning. Optimised for a Command Line
Interface (CLI), BookBob allows for efficient management of patient information and appointments.
allergies, medical histories and visit records with details on diagnoses and prescribed medications. Through the usage
of BookBob, users will also be able to schedule appointments with their patients and receive daily reminder of the
appointments for the day upon login. The application is optimised for Command Line Interface (CLI), allowing for
efficient management of patient information and appointments.

### Summary of Contributions
- Code Contributed: [RepoSense Link](https://nus-cs2113-ay2425s1.github.io/tp-dashboard/?search=g13nd0n)
@@ -24,12 +25,13 @@ Interface (CLI), BookBob allows for efficient management of patient information
- Furthermore, once added the appointment, the list of appointments will be sorted according to date and time.
Implementation required ensure the appointment class implements the Comparable interface and have the compareTo
methods and sorting them out. Additionally, implementation will need to ensure that regardless the order of the
appointment is added, it will be sorted appropriately
appointment is added, it will be sorted appropriately after the appointment has been added.
while comparison are correct
- Delete appointments
- What it does: delete appointments based on the specific nric, date and time given
- Justification: This feature improves product by allowing user to delete the appointments that might have been
made on accident. It requires date and time to ensure that the appointment deleted is the correct appointment.
The delete feature is case-insensitive for the nric when deleting the appointment.
- List all appointments
- What it does: List all appointments in chronological order
- Justification: This improves the product by allowing user to look at all the appointments that has been made
@@ -48,7 +50,7 @@ Interface (CLI), BookBob allows for efficient management of patient information
- Justification: This feature improves the product by removing the need for user to personally delete the
appointments that have already passed.
- Highlights: The appointments that are removed would be those that are before the time that the user logged off.
Implementations would required to check both date and time before removal and not simply removing all the
Implementations would be required to check both date and time before removal and not simply removing all the
appointments for the day.
- Wrote additional testing for the new features added. Included testing of methods defined in the `Appointment` and
`AppointmentRecord` classes. #248
@@ -59,17 +61,25 @@ Interface (CLI), BookBob allows for efficient management of patient information
- Added class diagram for `Appointment` and `AppointmentRecord` class
- Added details, sequence diagram and object diagram for the Appointment Feature
- Added information for the Glossary
- Check Contributions to Developer Guide for UML diagrams
- Contributions to team-based task :
- Convert the extraction of patient's details into a method since it has been used repeatedly #343
- Remove all the magic number when extracting patient's details #343
- Maintain the issue tracker
- Review/mentoring contributions:
- Help teammates in solving JUnit test issues e.g. System.lineSeparator() was not replaced with \n
- Help fixed the CI issues that was accumulated over time #75
- Contributions beyond the project team:
- Help to spot bugs during dry run practical
- Peer review given to certain groups regarding their product and Developer Guide

## Contributions to the Developer Guide
- Appointment Class Diagram\
![Appointment.png](..%2FAppointment.png)
- AppointmentRecord Class Diagram\
![AppointmentRecord.png](..%2FAppointmentRecord.png)
- Object Diagram for Appointment Feature\
![BeforeAppointmentOD.png](..%2FBeforeAppointmentOD.png)
- ![BeforeAppointmentOD.png](..%2FBeforeAppointmentOD.png)
![AfterAppointmentOD.png](..%2FAfterAppointmentOD.png)
- Sequence Diagram for Appointment Feature\
![NewAppointmentSD.png](..%2FNewAppointmentSD.png)