Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
516 commits
Select commit Hold shift + click to select a range
0122b0e
Merge pull request #146 from Phua-Lock-Hian/lockhian-fixDGissues
Phua-Lock-Hian Apr 3, 2025
7af9949
fix feedback #143
Phua-Lock-Hian Apr 3, 2025
f7676b5
fix feedback #141
Phua-Lock-Hian Apr 3, 2025
635e6e3
Merge pull request #147 from Phua-Lock-Hian/lockhian-fixDG2
Phua-Lock-Hian Apr 3, 2025
d668e5d
Rename addloan.puml to addLoan.puml
ShengBin-101 Apr 3, 2025
960cb07
Merge branch 'master' of https://github.com/AY2425S2-CS2113-T12-2/tp …
ShengBin-101 Apr 3, 2025
9fb6456
fix naming for FileWriter object and saveInventory Sequence Diagram
ShengBin-101 Apr 3, 2025
e40619c
Fix instantiation and destroy File and FileWriter
ShengBin-101 Apr 3, 2025
66dd6b5
Merge pull request #148 from ShengBin-101/shengbin-DG-week11-fixes
ShengBin-101 Apr 3, 2025
2c72f16
Add expected outcomes
dylankhoo Apr 3, 2025
9a9496f
Merge pull request #149 from dylankhoo/update-user-guide
dylankhoo Apr 3, 2025
d2b67c2
Update break 1
dylankhoo Apr 3, 2025
506d199
fix feedback #140
Phua-Lock-Hian Apr 3, 2025
081f034
fix feedback #140
Phua-Lock-Hian Apr 3, 2025
9257b6a
Merge remote-tracking branch 'origin/lockhian-fixDG3' into lockhian-f…
Phua-Lock-Hian Apr 3, 2025
deb8248
Fix loadInventory sequence diagram
ShengBin-101 Apr 3, 2025
5e8333e
Merge branch 'master' of https://github.com/AY2425S2-CS2113-T12-2/tp …
ShengBin-101 Apr 3, 2025
b7e0d80
Fix loadInventory sequence diagram and explanation
ShengBin-101 Apr 3, 2025
aa3a89e
fix feedback #138
Phua-Lock-Hian Apr 3, 2025
ee80622
Merge pull request #150 from ShengBin-101/shengbin-DG-week11-fixes
ShengBin-101 Apr 3, 2025
8d47b26
fix feedback #138
Phua-Lock-Hian Apr 3, 2025
fef2b94
edit via MS paint
Phua-Lock-Hian Apr 3, 2025
104bf7e
Merge remote-tracking branch 'origin/lockhian-fixDG3' into lockhian-f…
Phua-Lock-Hian Apr 3, 2025
f4c83e2
put back attributes for Storage
Phua-Lock-Hian Apr 3, 2025
57dba87
edit via MS paint again
Phua-Lock-Hian Apr 3, 2025
7c0960e
Merge pull request #151 from Phua-Lock-Hian/lockhian-fixDG3
Phua-Lock-Hian Apr 3, 2025
fd409c8
Add else to alt blocks
ShengJie13245 Apr 3, 2025
ee64cd2
change alt to opt for if blocks
ShengBin-101 Apr 3, 2025
eed41c2
Merge pull request #152 from ShengJie13245/ShengJieDG
ShengJie13245 Apr 3, 2025
3722852
Merge pull request #153 from ShengBin-101/shengbin-DG-week11-fixes
ShengBin-101 Apr 3, 2025
fcf4a53
Change alt to opt in loadInventory Sequence Diagram
ShengBin-101 Apr 3, 2025
f4642df
Update loadInventory.png
ShengBin-101 Apr 3, 2025
aece9c6
fix activation bar before final return
Phua-Lock-Hian Apr 3, 2025
1bf3860
Merge pull request #154 from ShengBin-101/shengbin-DG-week11-fixes
ShengBin-101 Apr 3, 2025
355bbd0
add else to alt blocks
Phua-Lock-Hian Apr 3, 2025
de62f11
overwrite pngs
Phua-Lock-Hian Apr 3, 2025
abaad09
Add [else] for else branch in sequence diagram
ShengBin-101 Apr 3, 2025
abeed99
Merge pull request #155 from Phua-Lock-Hian/lockhian-fixDG4
ShengBin-101 Apr 3, 2025
a373747
Merge branch 'master' of https://github.com/AY2425S2-CS2113-T12-2/tp …
ShengBin-101 Apr 3, 2025
d4f9748
Merge pull request #156 from ShengBin-101/shengbin-DG-week11-fixes
ShengBin-101 Apr 3, 2025
9a48222
Add [else] for else branch in sequence diagram
ShengBin-101 Apr 3, 2025
6daf5d3
Merge pull request #157 from ShengBin-101/shengbin-DG-week11-fixes
ShengBin-101 Apr 3, 2025
f7ac47d
Update to viewInventory and viewLoans UMLs
ShengJie13245 Apr 3, 2025
b2d36c9
Merge pull request #158 from ShengJie13245/ShengJieDG
ShengJie13245 Apr 3, 2025
c7578c7
Update sequence diagram for addBook, addLoan
dylankhoo Apr 3, 2025
797a212
Fix removeBook.puml to include arrow back to User
ShengBin-101 Apr 3, 2025
67528f2
Merge branch 'master' of https://github.com/AY2425S2-CS2113-T12-2/tp …
ShengBin-101 Apr 3, 2025
2e1b989
Merge pull request #159 from dylankhoo/update-developer-guide
dylankhoo Apr 3, 2025
dbb321a
Merge pull request #161 from ShengBin-101/shengbin-DG-week11-fixes
ShengBin-101 Apr 3, 2025
6e17842
Update else block in sequence diagram
dylankhoo Apr 3, 2025
5d533d4
Update UML diagrams for viewInventory, viewLoans
ShengJie13245 Apr 3, 2025
58daad6
Update else block in PlantUML
dylankhoo Apr 3, 2025
55b30bb
Merge pull request #162 from dylankhoo/update-developer-guide
dylankhoo Apr 3, 2025
67a7422
Merge branch 'master' of https://github.com/AY2425S2-CS2113-T12-2/tp …
ShengJie13245 Apr 3, 2025
550e526
Merge pull request #163 from ShengJie13245/ShengJieDG
ShengJie13245 Apr 3, 2025
8606aef
Update deleteLoan, removeBook diagram
dylankhoo Apr 3, 2025
5aefd72
Merge pull request #164 from dylankhoo/update-developer-guide
dylankhoo Apr 3, 2025
5781f51
Merge branch 'master' of https://github.com/AY2425S2-CS2113-T12-2/tp …
dylankhoo Apr 3, 2025
6ea118f
Merge pull request #165 from dylankhoo/update-user-guide
dylankhoo Apr 3, 2025
1c1bfd9
Add Placeholder Portrait for AboutUs.md
ShengBin-101 Apr 5, 2025
e59564a
Merge branch 'master' of https://github.com/AY2425S2-CS2113-T12-2/tp …
ShengBin-101 Apr 5, 2025
6a3a9e7
Fix link to v2.0 Jar
ShengBin-101 Apr 5, 2025
5532324
Merge pull request #204 from ShengBin-101/shengbin-PED-fixes
ShengBin-101 Apr 5, 2025
eee0a76
Fix Inconsistent Book Conditions in DG
ShengBin-101 Apr 5, 2025
f85819f
Merge pull request #205 from ShengBin-101/shengbin-PED-fixes
ShengBin-101 Apr 5, 2025
11b4ffd
fix #203
Phua-Lock-Hian Apr 5, 2025
b64e440
fix #178
Phua-Lock-Hian Apr 5, 2025
60b6f54
fix #190
Phua-Lock-Hian Apr 5, 2025
75ecc4c
Implement Date Validation using DateTimeFormatter
ShengBin-101 Apr 5, 2025
21573f5
Implement Date Validation using DateTimeFormatter
ShengBin-101 Apr 5, 2025
2f6ac89
Merge branch 'master' of https://github.com/AY2425S2-CS2113-T12-2/tp …
ShengBin-101 Apr 5, 2025
3fe5ef3
Merge pull request #206 from ShengBin-101/shengbin-PED-fixes
ShengBin-101 Apr 5, 2025
5f89648
Fix loading of loans
ShengBin-101 Apr 5, 2025
08f36c7
Case-insensitive title search and loading
ShengBin-101 Apr 5, 2025
6f6fb2d
edit EXPECTED.TXT to reflect change to delete-loan
Phua-Lock-Hian Apr 5, 2025
15fb311
remove unused import in BookList
Phua-Lock-Hian Apr 5, 2025
7e352de
Merge pull request #208 from Phua-Lock-Hian/lockhian-pedfixes
Phua-Lock-Hian Apr 5, 2025
51c582e
Fix add-loan with empty parameters
ShengBin-101 Apr 5, 2025
e190fdb
Merge branch 'master' of https://github.com/AY2425S2-CS2113-T12-2/tp …
ShengBin-101 Apr 5, 2025
0ec18ba
Merge pull request #207 from ShengBin-101/shengbin-PED-fixes
ShengBin-101 Apr 5, 2025
69075f7
Fix InputParser to handle missing/blank parameters
ShengBin-101 Apr 5, 2025
3b79d8e
Merge branch 'master' of https://github.com/AY2425S2-CS2113-T12-2/tp …
ShengBin-101 Apr 5, 2025
c4ca72c
Merge pull request #209 from ShengBin-101/shengbin-PED-fixes
ShengBin-101 Apr 5, 2025
6b07963
Update viewLoans Sequence Diagram to include [else]
ShengBin-101 Apr 5, 2025
1d15f49
Merge branch 'master' of https://github.com/AY2425S2-CS2113-T12-2/tp …
ShengBin-101 Apr 5, 2025
f870768
Update viewLoans Sequence Diagram to include [else]
ShengBin-101 Apr 5, 2025
3a705b8
Merge pull request #210 from ShengBin-101/shengbin-PED-fixes
ShengBin-101 Apr 5, 2025
aa20c7d
disallow | to be used in user input since it is used as separator for…
Phua-Lock-Hian Apr 5, 2025
5703c8b
update EXPECTED.TXT
Phua-Lock-Hian Apr 5, 2025
0715c3e
Merge pull request #211 from Phua-Lock-Hian/lockhian-pedfix_illegalchar
Phua-Lock-Hian Apr 5, 2025
ff0c315
added update-note function
Phua-Lock-Hian Apr 5, 2025
eacf282
Merge pull request #212 from Phua-Lock-Hian/lockhian-updatenote
Phua-Lock-Hian Apr 5, 2025
acb20fa
Add validation for phone number and email in edit and add loan
ShengJie13245 Apr 5, 2025
79e86ac
Merge pull request #213 from ShengJie13245/ShengJieValidPhone
ShengJie13245 Apr 5, 2025
1e75023
Update edit loan to have optional inputs
ShengJie13245 Apr 5, 2025
d8c6d8e
Edit UG,DG to match current operation
ShengJie13245 Apr 5, 2025
e7427e8
Edit to UG example
ShengJie13245 Apr 5, 2025
ca5b3b8
Add UG,DG for update-note
ShengJie13245 Apr 5, 2025
460219f
Edit to meet checkStyle
ShengJie13245 Apr 5, 2025
bd6409a
Merge pull request #215 from ShengJie13245/ShengJieDynamEditLoan
ShengJie13245 Apr 5, 2025
c9e26b8
Merge pull request #216 from ShengJie13245/ShengJieUG
ShengJie13245 Apr 5, 2025
091d138
Add DG for add and delete note
ShengJie13245 Apr 5, 2025
c574d27
Merge pull request #218 from ShengJie13245/ShengJieDG
ShengJie13245 Apr 5, 2025
b4f8c36
fix #219
Phua-Lock-Hian Apr 5, 2025
4fef692
Remove duplicate section for update-note in DG
ShengBin-101 Apr 6, 2025
1fa8a90
Merge pull request #221 from ShengBin-101/shengbin-PED-fixes
ShengBin-101 Apr 6, 2025
50f5244
Change book category from string to enum
ShengBin-101 Apr 6, 2025
a5de7e4
Merge pull request #222 from ShengBin-101/shengbin-PED-fixes
ShengBin-101 Apr 6, 2025
f891391
testing one diagram first waiting for team confirmation
Phua-Lock-Hian Apr 6, 2025
b5c90e5
own PPP edits
Phua-Lock-Hian Apr 6, 2025
fcd1367
addBook and addLoan sequence diagrams reduced nesting
Phua-Lock-Hian Apr 6, 2025
be4ee47
Update view-inventory for books on loan
dylankhoo Apr 6, 2025
703d5b5
Merge pull request #223 from dylankhoo/update-view-inventory
dylankhoo Apr 6, 2025
b367f77
Make update-book fields optional
dylankhoo Apr 6, 2025
e073f8d
Refactor setLoanFields, setBookFields, to Loan and Book class
dylankhoo Apr 6, 2025
fec3c19
Add update-title command
dylankhoo Apr 6, 2025
c4fa0f5
Add exceptions, comments for update-title
dylankhoo Apr 6, 2025
d469cc5
Remove add-note, update-note, delete-note commands
dylankhoo Apr 6, 2025
4ffe0f5
reduced nesting
Phua-Lock-Hian Apr 6, 2025
fbbff08
fix incomplete integration of change to delete-loan to not require bo…
Phua-Lock-Hian Apr 6, 2025
1e5dcc2
Fix case for empty fields in update-book, edit-loan
dylankhoo Apr 6, 2025
e795971
Update UG for update-title, note commands, update-book, edit-loan
dylankhoo Apr 6, 2025
9cfca03
sequence diagram edits
Phua-Lock-Hian Apr 6, 2025
4835292
Update test, code formatting
dylankhoo Apr 6, 2025
545f72d
sequence diagram edits
Phua-Lock-Hian Apr 6, 2025
82c0e46
Merge remote-tracking branch 'origin/lockhian-ugconstraints_dgoptblks…
Phua-Lock-Hian Apr 6, 2025
54c5fbf
Merge pull request #224 from dylankhoo/refactor-update-book
dylankhoo Apr 6, 2025
2510a50
Merge branch 'master' into lockhian-ugconstraints_dgoptblks
Phua-Lock-Hian Apr 6, 2025
5f368be
Merge pull request #225 from Phua-Lock-Hian/lockhian-ugconstraints_dg…
Phua-Lock-Hian Apr 6, 2025
391f61e
add InputParser tests
Phua-Lock-Hian Apr 6, 2025
a34edbe
add LoanList tests
Phua-Lock-Hian Apr 6, 2025
2317b18
add LoanList tests
Phua-Lock-Hian Apr 6, 2025
0a39f6b
Merge remote-tracking branch 'origin/lockhian-addtests' into lockhian…
Phua-Lock-Hian Apr 6, 2025
03503ad
Merge pull request #226 from Phua-Lock-Hian/lockhian-addtests
Phua-Lock-Hian Apr 6, 2025
4bd26f8
minor update to PPP
Phua-Lock-Hian Apr 6, 2025
ba6a984
Bug Fixes
ShengJie13245 Apr 6, 2025
0e15485
Edit to fit checkStyle
ShengJie13245 Apr 6, 2025
4b89ecc
ErrorMessage string fix to show correct message
ShengJie13245 Apr 6, 2025
eeefe22
Edit to fit checkStyle
ShengJie13245 Apr 6, 2025
2d1ad59
Edit to fit checkStyle
ShengJie13245 Apr 6, 2025
374eb33
Edit to fit checkStyle
ShengJie13245 Apr 6, 2025
088f5d7
Edit to fit checkStyle
ShengJie13245 Apr 6, 2025
87508cc
Edit to fit checkStyle
ShengJie13245 Apr 6, 2025
75e8396
Merge pull request #227 from ShengJie13245/ShengJieBugFix
ShengJie13245 Apr 6, 2025
bceb2e3
Update classDiagram.png and fix brace position in Storage.java
ShengBin-101 Apr 6, 2025
0dacd7d
Update date validation to handle leap year and 31st of months with 30…
ShengBin-101 Apr 6, 2025
fab241c
Merge branch 'master' of https://github.com/AY2425S2-CS2113-T12-2/tp …
ShengBin-101 Apr 6, 2025
a599754
Merge pull request #228 from ShengBin-101/shengbin-PED-fixes
ShengBin-101 Apr 6, 2025
9480e51
Fix indentation
ShengBin-101 Apr 6, 2025
9facbf3
Merge pull request #229 from ShengBin-101/shengbin-PED-fixes
ShengBin-101 Apr 6, 2025
7b39626
Fix loading loans not checking for invalid inputs
ShengJie13245 Apr 6, 2025
35984dc
Fix addLoan not throwing correct exception
ShengJie13245 Apr 6, 2025
501e1a8
Edit to fit checkStyle
ShengJie13245 Apr 6, 2025
197e773
Edit to fit checkStyle
ShengJie13245 Apr 6, 2025
9e55899
Edit to fit checkStyle
ShengJie13245 Apr 6, 2025
cfc90e1
Merge pull request #230 from ShengJie13245/ShengJieDynamEditLoan
ShengJie13245 Apr 6, 2025
d5792ef
fix typo
Phua-Lock-Hian Apr 7, 2025
dbf31e6
clarify that phone numbers are singapore numbers only and change sear…
Phua-Lock-Hian Apr 7, 2025
408f94d
clarify that phone numbers are singapore numbers only and change sear…
Phua-Lock-Hian Apr 7, 2025
5311bc4
Merge remote-tracking branch 'origin/lockhian-ugfixes' into lockhian-…
Phua-Lock-Hian Apr 7, 2025
d51253a
update DG
Phua-Lock-Hian Apr 7, 2025
f850f18
enforce 8 digit and begin with 9 8 or 6 requirement for phone numbers
Phua-Lock-Hian Apr 7, 2025
179c195
Merge branch 'master' into lockhian-ugfixes
Phua-Lock-Hian Apr 7, 2025
1a430ca
Merge branch 'master' into lockhian-ugfixes
Phua-Lock-Hian Apr 7, 2025
7c7062e
Merge remote-tracking branch 'origin/lockhian-ugfixes' into lockhian-…
Phua-Lock-Hian Apr 7, 2025
775675b
typo fix
Phua-Lock-Hian Apr 7, 2025
d8847fe
typo fix
Phua-Lock-Hian Apr 7, 2025
f9ca2d1
javadoc edit
Phua-Lock-Hian Apr 7, 2025
26d2371
edit tests to reflect change in constraints to phone nums
Phua-Lock-Hian Apr 7, 2025
b417fa4
Merge pull request #231 from Phua-Lock-Hian/lockhian-ugfixes
Phua-Lock-Hian Apr 7, 2025
886f9cd
Add update to storage for phone number
ShengJie13245 Apr 7, 2025
7a7fb2e
Fix bug for duplicate title in update-book
dylankhoo Apr 7, 2025
928890b
Update help command display
dylankhoo Apr 7, 2025
183291f
Edit to check for correct phone number format in loadloans
ShengJie13245 Apr 7, 2025
4034f34
Merge pull request #232 from ShengJie13245/ShengJieDynamEditLoan
ShengJie13245 Apr 7, 2025
5cda1ed
Add tests for update-title
dylankhoo Apr 7, 2025
af02d52
Update runtest
dylankhoo Apr 7, 2025
d7950d3
Remove whitespace
dylankhoo Apr 7, 2025
9a02cd6
Add newline to expected.txt
dylankhoo Apr 7, 2025
ba97896
Merge pull request #233 from dylankhoo/refactor-update-book
dylankhoo Apr 7, 2025
f4deb48
Remove outdated note commands
dylankhoo Apr 7, 2025
69d1b56
Add some JUnit tests
ShengJie13245 Apr 7, 2025
72a0141
Merge branch 'master' of https://github.com/AY2425S2-CS2113-T12-2/tp …
ShengJie13245 Apr 7, 2025
ecccef0
Update sequence diagram for update-book
dylankhoo Apr 7, 2025
2d27049
Edit to fit checkStyle
ShengJie13245 Apr 7, 2025
a8b1a3f
Merge pull request #234 from ShengJie13245/ShengJie-JUnit
ShengJie13245 Apr 7, 2025
13334cb
Fix bug for update-title
dylankhoo Apr 7, 2025
bab5af6
change classDiagram.png to have the notes linked to the classes corre…
Phua-Lock-Hian Apr 7, 2025
35447d7
Update DG for update-title
dylankhoo Apr 7, 2025
06b38fa
Update to DG and PPP
ShengJie13245 Apr 7, 2025
41b4750
Merge pull request #236 from ShengJie13245/ShengJiePPP
ShengJie13245 Apr 7, 2025
afdbb5a
Merge pull request #237 from dylankhoo/update-developer-guide
dylankhoo Apr 7, 2025
b01c96b
Update PPP
ShengJie13245 Apr 7, 2025
17345ef
edit success message printing in sequence diagram for addLoan to refl…
Phua-Lock-Hian Apr 7, 2025
089ce0f
edit success message printing in sequence diagram for removebook to r…
Phua-Lock-Hian Apr 7, 2025
9ac8438
edit success message printing in sequence diagram for removebook to r…
Phua-Lock-Hian Apr 7, 2025
9a66bbf
Merge remote-tracking branch 'origin/lockhian-docscheck' into lockhia…
Phua-Lock-Hian Apr 7, 2025
af08a0e
edit success message printing in sequence diagram for updatebook to r…
Phua-Lock-Hian Apr 7, 2025
04bb129
edit updatebook sequence diagram typo
Phua-Lock-Hian Apr 7, 2025
c95e361
Merge branch 'master' into ShengJiePPP
ShengJie13245 Apr 7, 2025
ff2dee0
Edit PPP to fit page limit
ShengJie13245 Apr 7, 2025
75eb729
Merge pull request #238 from ShengJie13245/ShengJiePPP
ShengJie13245 Apr 7, 2025
799e68a
Merge branch 'lockhian-docscheck'
Phua-Lock-Hian Apr 7, 2025
0d0242a
Merge pull request #235 from Phua-Lock-Hian/lockhian-docscheck
Phua-Lock-Hian Apr 7, 2025
61ec72c
Edit PPP
ShengJie13245 Apr 7, 2025
f03b7a1
Merge pull request #239 from ShengJie13245/ShengJiePPP
ShengJie13245 Apr 7, 2025
a8c0614
Edit update-book to search book case-sensitive
ShengJie13245 Apr 7, 2025
31a68a0
Edit DG/UG
ShengJie13245 Apr 7, 2025
ac08dbf
Merge branch 'master' of https://github.com/AY2425S2-CS2113-T12-2/tp …
ShengJie13245 Apr 7, 2025
f0c39f0
Edit DG/UG
ShengJie13245 Apr 7, 2025
7b5606b
Merge pull request #240 from ShengJie13245/ShengJiePPP
ShengJie13245 Apr 7, 2025
ffe7dab
Update dylan-PPP
dylankhoo Apr 7, 2025
0abd3e0
Add back delete-note
dylankhoo Apr 7, 2025
9793232
Merge pull request #241 from dylankhoo/delete-note
dylankhoo Apr 7, 2025
4b5b3f8
Merge pull request #242 from dylankhoo/dylan-PPP
dylankhoo Apr 7, 2025
ae83106
Fix InputParser to handle names with "s/o"
ShengBin-101 Apr 7, 2025
a65a554
Fix javadoc
ShengBin-101 Apr 7, 2025
735033d
Update UG page break 1
dylankhoo Apr 7, 2025
ae493b8
Merge pull request #244 from dylankhoo/update-page-break
dylankhoo Apr 7, 2025
1e098fb
Fix UG command table and include description for valid phone numbers …
ShengBin-101 Apr 7, 2025
1c69735
Fix Merge Conflicts
ShengBin-101 Apr 7, 2025
4a15287
Merge pull request #243 from ShengBin-101/shengbin-PED-fixes
ShengBin-101 Apr 7, 2025
e154ccb
Update DG to use searchBook
dylankhoo Apr 7, 2025
fa1e65e
Fix update-title to use case-sensitive comparison
ShengBin-101 Apr 7, 2025
046c98c
Merge branch 'master' of https://github.com/AY2425S2-CS2113-T12-2/tp …
ShengBin-101 Apr 7, 2025
65b3dfd
Rename searchBook() in InputHandler to searchTitle()
ShengBin-101 Apr 7, 2025
1f2533c
Update all sequence diagrams to use searchBook
dylankhoo Apr 7, 2025
71f2876
Add sequence diagram for search-title
dylankhoo Apr 7, 2025
d7715a6
Update DG to include loadLoans and saveLoans
ShengBin-101 Apr 7, 2025
70bbe75
Merge pull request #245 from ShengBin-101/shengbin-PED-fixes
ShengBin-101 Apr 7, 2025
a558fb7
Merge branch 'master' into update-developer-guide
dylankhoo Apr 7, 2025
3e80b35
Merge pull request #246 from dylankhoo/update-developer-guide
dylankhoo Apr 7, 2025
f9a5efe
Search-title Sequence Diagram in DG
ShengBin-101 Apr 7, 2025
77a6c10
Merge pull request #247 from ShengBin-101/shengbin-PED-fixes
ShengBin-101 Apr 7, 2025
ec6e09a
Update missing info
dylankhoo Apr 7, 2025
267cc26
Merge pull request #248 from dylankhoo/update-user-guide
dylankhoo Apr 7, 2025
7559a6e
add commandArgs into the method calls as arguments to reflect actual …
Phua-Lock-Hian Apr 8, 2025
1f6bcaf
Merge pull request #249 from Phua-Lock-Hian/lockhian-finalDGchecks
Phua-Lock-Hian Apr 8, 2025
0d14377
Update shengbin PPP
ShengBin-101 Apr 8, 2025
9b24208
Merge pull request #250 from ShengBin-101/shengbin-PPP
ShengBin-101 Apr 8, 2025
649232d
Update dylan-PPP
dylankhoo Apr 8, 2025
24861f8
Merge pull request #251 from dylankhoo/dylan-PPP
dylankhoo Apr 8, 2025
cc39095
Update dylan-PPP for page limit
dylankhoo Apr 8, 2025
1238922
Merge pull request #252 from dylankhoo/dylan-PPP
dylankhoo Apr 8, 2025
3563f72
Update dylan-PPP for page length constraints
dylankhoo Apr 8, 2025
e6584a0
Merge pull request #253 from dylankhoo/dylan-PPP
dylankhoo Apr 8, 2025
83edb52
Edit PPP shengbin-101.md
ShengBin-101 Apr 8, 2025
3b832ef
Merge pull request #254 from ShengBin-101/shengbin-PPP
ShengBin-101 Apr 8, 2025
514af59
Edit PPP shengbin-101.md, to fit to 2 pages
ShengBin-101 Apr 8, 2025
508e6e2
Merge pull request #255 from ShengBin-101/shengbin-PPP
ShengBin-101 Apr 8, 2025
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
7 changes: 4 additions & 3 deletions .github/workflows/gradle.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,11 +22,12 @@ jobs:
run: git checkout --progress --force ${{ github.sha }}

- name: Validate Gradle Wrapper
uses: gradle/wrapper-validation-action@v1
uses: gradle/actions/wrapper-validation@v3

- name: Setup JDK 17
uses: actions/setup-java@v1
uses: actions/setup-java@v4
with:
distribution: 'zulu'
java-version: '17'
java-package: jdk+fx

Expand All @@ -47,4 +48,4 @@ jobs:
if: always() && runner.os == 'Windows'
working-directory: ${{ github.workspace }}/text-ui-test
shell: cmd
run: runtest.bat
run: runtest.bat
6 changes: 6 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@
/out/
/*.iml

*.class

# Gradle build files
/.gradle/
/build/
Expand All @@ -15,3 +17,7 @@ bin/

/text-ui-test/ACTUAL.TXT
text-ui-test/EXPECTED-UNIX.TXT
bookkeeper.log
bookkeeper.log.lck
/data/bookKeeper_bookList.txt
/data/bookKeeper_loanList.txt
26 changes: 5 additions & 21 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,30 +1,14 @@
# Duke project template
# BookKeeper project template

This is a project template for a greenfield Java project. It's named after the Java mascot _Duke_. Given below are instructions on how to use it.
BookKeeper is a Command Line Interface (CLI) library manager application for effective tracking of library loans and inventory. It allows users to manage books, loans, and notes efficiently through a set of commands.

## Setting up in Intellij

Prerequisites: JDK 17 (use the exact version), update Intellij to the most recent version.

1. **Ensure Intellij JDK 17 is defined as an SDK**, as described [here](https://www.jetbrains.com/help/idea/sdk.html#set-up-jdk) -- this step is not needed if you have used JDK 17 in a previous Intellij project.
1. **Import the project _as a Gradle project_**, as described [here](https://se-education.org/guides/tutorials/intellijImportGradleProject.html).
1. **Verify the setup**: After the importing is complete, locate the `src/main/java/seedu/duke/Duke.java` file, right-click it, and choose `Run Duke.main()`. If the setup is correct, you should see something like the below:
```
> Task :compileJava
> Task :processResources NO-SOURCE
> Task :classes

> Task :Duke.main()
Hello from
____ _
| _ \ _ _| | _____
| | | | | | | |/ / _ \
| |_| | |_| | < __/
|____/ \__,_|_|\_\___|

What is your name?
```
Type some word and press enter to let the execution proceed to the end.
2. **Import the project _as a Gradle project_**, as described [here](https://se-education.org/guides/tutorials/intellijImportGradleProject.html).
3. **Verify the setup**: After the importing is complete, locate the `src/main/java/BookKeeper.java` file, right-click it, and choose `Run BookKeeper.main()`.

**Warning:** Keep the `src\main\java` folder as the root folder for Java files (i.e., don't rename those folders or move Java files to another folder outside of this folder path), as this is the default location some tools (e.g., Gradle) expect to find Java files.

Expand All @@ -41,7 +25,7 @@ Prerequisites: JDK 17 (use the exact version), update Intellij to the most recen

### JUnit tests

* A skeleton JUnit test (`src/test/java/seedu/duke/DukeTest.java`) is provided with this project template.
* A skeleton JUnit test (`src/test/java/DukeTest.java`) is provided with this project template.
* If you are new to JUnit, refer to the [JUnit Tutorial at se-education.org/guides](https://se-education.org/guides/tutorials/junit.html).

## Checkstyle
Expand Down
5 changes: 3 additions & 2 deletions build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -29,11 +29,11 @@ test {
}

application {
mainClass.set("seedu.duke.Duke")
mainClass.set("bookkeeper.BookKeeper")
}

shadowJar {
archiveBaseName.set("duke")
archiveBaseName.set("BookKeeper")
archiveClassifier.set("")
}

Expand All @@ -42,5 +42,6 @@ checkstyle {
}

run{
enableAssertions = true
standardInput = System.in
}
14 changes: 7 additions & 7 deletions docs/AboutUs.md
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
# About us

Display | Name | Github Profile | Portfolio
--------|:----:|:--------------:|:---------:
![](https://via.placeholder.com/100.png?text=Photo) | John Doe | [Github](https://github.com/) | [Portfolio](docs/team/johndoe.md)
![](https://via.placeholder.com/100.png?text=Photo) | Don Joe | [Github](https://github.com/) | [Portfolio](docs/team/johndoe.md)
![](https://via.placeholder.com/100.png?text=Photo) | Ron John | [Github](https://github.com/) | [Portfolio](docs/team/johndoe.md)
![](https://via.placeholder.com/100.png?text=Photo) | John Roe | [Github](https://github.com/) | [Portfolio](docs/team/johndoe.md)
![](https://via.placeholder.com/100.png?text=Photo) | Don Roe | [Github](https://github.com/) | [Portfolio](docs/team/johndoe.md)
| Display | Name | Github Profile | Portfolio |
|-----------------------------------------------------|:--------------:| :------------------------------------: |:-----------------------------------:|
| ![](images/Portrait_placeholder.png) | Dylan Khoo | [Github](https://github.com/dylankhoo) | [Portfolio](team/dylankhoo.md) |
| ![](images/Portrait_placeholder.png) | Chan Sheng Bin | [Github](https://github.com/ShengBin-101) | [Portfolio](team/shengbin-101.md) |
![](images/Portrait_placeholder.png) | Phua Lock Hian | [Github](https://github.com/phua-lock-hian) | [Portfolio](team/phua-lock-hian.md) |
![](images/Portrait_placeholder.png) | Kam Sheng Jie | [Github](https://github.com/ShengJie13245) | [Portfolio](team/shengjie13245.md)

Loading