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-W12-1] Upcycle #3

Open
wants to merge 602 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from 250 commits
Commits
Show all changes
602 commits
Select commit Hold shift + click to select a range
2acce21
Merge pull request #130 from bdthanh/branch-storage-illegally-modified
bdthanh Oct 26, 2022
59db884
Merge pull request #129 from bdthanh/branch-edit-dg
bdthanh Oct 26, 2022
89d8b5d
Added find-item to UG
jorellesee Oct 26, 2022
8fe9555
Merge pull request #131 from jorellesee/master
jorellesee Oct 26, 2022
91f8c93
Fix bugs
bdthanh Oct 26, 2022
9e76a25
Merge pull request #132 from bdthanh/branch-storage-illegally-modified
bdthanh Oct 26, 2022
63b9669
Fix bug when price > 99999999999 (limit range 0-10000)
bdthanh Oct 27, 2022
371aa56
Merge branch 'master' of https://github.com/AY2223S1-CS2113-W12-1/tp …
bdthanh Oct 27, 2022
e04b993
Update UG
bdthanh Oct 27, 2022
21b20d0
Merge pull request #133 from bdthanh/branch-bug-price-overflows
bdthanh Oct 27, 2022
e000ba7
Limit range of price, duration, createdDate, itemname, username lengt…
bdthanh Oct 27, 2022
b654ecf
Fix checkstyle
bdthanh Oct 27, 2022
d32adbd
Merge pull request #134 from bdthanh/branch-limit-arg-range
bdthanh Oct 27, 2022
cabc4e4
Update UserGuide.md
jingwei55 Oct 27, 2022
55b9198
Added new methods for ViewUserDebt and to FindTransactionsByUser
jorellesee Oct 27, 2022
3f74674
Merge pull request #135 from jorellesee/master
bdthanh Oct 27, 2022
63c56f9
Clean code
bdthanh Oct 27, 2022
c57e4f3
Fix UG
bdthanh Oct 27, 2022
bc40258
Merge pull request #136 from bdthanh/branch-clean
bdthanh Oct 27, 2022
826e698
Update DG
bdthanh Oct 28, 2022
9305f65
Update TransactionClassDiagram
bdthanh Oct 28, 2022
f20cf03
added more JUnit tests for User and UserList class
chiewyx Oct 28, 2022
4eaaaa1
added more javadoc for user and userlist class
chiewyx Oct 28, 2022
f1e8698
Fix UG typo and grammars and bug in add-tx command
bdthanh Oct 28, 2022
c62e2ad
Merge pull request #185 from bdthanh/branch-ped
bdthanh Oct 28, 2022
16868e5
Merge branch 'master' of https://github.com/AY2223S1-CS2113-W12-1/tp …
chiewyx Oct 29, 2022
8328b61
fix code for checkstyle
chiewyx Oct 29, 2022
8dfb1de
Merge pull request #187 from chiewyx/branch-tests
chiewyx Oct 29, 2022
c6568bb
added feature to view user items
chiewyx Oct 29, 2022
c1fb4f1
fix code for checkstyle
chiewyx Oct 29, 2022
89ec552
Add checksum to storage and fix transaction bug
bdthanh Oct 29, 2022
070b5b4
Added reminders in help command
jingwei55 Oct 29, 2022
bcb8cbc
Added reminders in help command
jingwei55 Oct 29, 2022
b971a32
Troubleshoot display
jingwei55 Oct 29, 2022
ce56270
Merge branch 'master' of https://github.com/AY2223S1-CS2113-W12-1/tp …
bdthanh Oct 29, 2022
c6d07bc
updated DG and UG for view-user and view-user-items command
chiewyx Oct 29, 2022
964e568
Modified to add item of same name BUT different price
jingwei55 Oct 29, 2022
7ea6a87
Update UserGuide.md
jingwei55 Oct 29, 2022
f152ec6
Update UserGuide.md
jingwei55 Oct 29, 2022
23bcab0
Change transactions to use BigDecimal instead, for proper double arit…
jingwei55 Oct 29, 2022
a742c7e
Update UserGuide.md
jingwei55 Oct 29, 2022
c177f68
Update UG and Fix storage bug
bdthanh Oct 29, 2022
144c3a4
Modified addUser to prevent duplicate contact number
jingwei55 Oct 29, 2022
e144f83
Some display changes, not sure why double arithmetic not solved with …
jingwei55 Oct 29, 2022
3676a28
Merge pull request #190 from jingwei55/master
jingwei55 Oct 29, 2022
f8f981e
Merge branch 'master' of https://github.com/AY2223S1-CS2113-W12-1/tp …
chiewyx Oct 29, 2022
ea9f772
Merge pull request #189 from chiewyx/branch-findUserItems
chiewyx Oct 29, 2022
cdeff23
Merge branch 'master' of https://github.com/AY2223S1-CS2113-W12-1/tp …
bdthanh Oct 29, 2022
8bb8803
Fixed bigdecimal, made price format be 2dp to look nicer
jingwei55 Oct 29, 2022
0e38e54
Finish Storage (OMG my whole day)
bdthanh Oct 29, 2022
a011531
troubleshoot checkstyle
jingwei55 Oct 29, 2022
65e4b34
troubleshoot checkstyle
jingwei55 Oct 29, 2022
4454d7d
Fix io check
bdthanh Oct 29, 2022
04b3df3
wtf Intellij say they are identical
jingwei55 Oct 29, 2022
85b992c
commented out the error
jingwei55 Oct 29, 2022
ee8a5cd
pls
jingwei55 Oct 29, 2022
25c1c1e
Merge pull request #193 from jingwei55/master
jingwei55 Oct 29, 2022
2b20a7c
Merge branch 'master' into branch-storge-checksum
bdthanh Oct 29, 2022
4bb9bbb
Merge branch 'master' of https://github.com/AY2223S1-CS2113-W12-1/tp …
bdthanh Oct 29, 2022
d03be64
Merge branch 'branch-storge-checksum' of https://github.com/bdthanh/t…
bdthanh Oct 29, 2022
1446680
Merge pull request #196 from bdthanh/branch-storge-checksum
bdthanh Oct 29, 2022
ef8c1f5
Fix UG
bdthanh Oct 29, 2022
2436f6a
Add PPP bdthanh.md
bdthanh Oct 29, 2022
b26e125
Fix test
bdthanh Oct 29, 2022
dbccef8
Merge pull request #197 from bdthanh/branch-ppp
bdthanh Oct 29, 2022
d4e659d
Update DeveloperGuide.md
jingwei55 Oct 30, 2022
af43a24
Create JingWei.md
jingwei55 Oct 30, 2022
7f4d4f7
Added ItemClassDiagram
jingwei55 Oct 30, 2022
4a6912a
Update and rename JingWei.md to jingwei.md
jingwei55 Oct 30, 2022
75ebda7
Merge pull request #198 from jingwei55/master
jingwei55 Oct 30, 2022
4f1ff4f
Update DeveloperGuide.md
jingwei55 Oct 30, 2022
9338384
Update DeveloperGuide.md
jingwei55 Oct 30, 2022
f61d6f9
Update jingwei.md
jingwei55 Oct 30, 2022
f898ab0
Update DeveloperGuide.md
jingwei55 Oct 30, 2022
594ca36
Rename jingwei.md to jingwei55.md
jingwei55 Oct 30, 2022
96f8d04
updated UG for view-user and view-user items
chiewyx Oct 30, 2022
45329de
updated user stories
chiewyx Oct 30, 2022
559ef27
updated help command to include view-user-items
chiewyx Oct 30, 2022
071ecc6
Merge pull request #202 from chiewyx/branch-UG
chiewyx Oct 30, 2022
1295839
added javadoc for ViewUserItemsCommand
chiewyx Oct 30, 2022
7b5ec31
Merge pull request #203 from chiewyx/branch-findUserItems
chiewyx Oct 30, 2022
556a4b1
Refactor magic literals and shorten methods' name
bdthanh Oct 30, 2022
65bc887
Merge pull request #210 from bdthanh/master
bdthanh Oct 30, 2022
3cdd1cb
added PPP
chiewyx Oct 31, 2022
692bc59
modified Sequence digram and deleted useless methods
jingwei55 Oct 31, 2022
475bbed
Merge branch 'master' of https://github.com/AY2223S1-CS2113-W12-1/tp
jingwei55 Oct 31, 2022
1b7e998
Remove magic literals and System.println
winston-lim Oct 31, 2022
6defced
Add HelpCommand test
winston-lim Oct 31, 2022
44dc3dd
Merge pull request #212 from winston-lim/master
winston-lim Oct 31, 2022
be40d9f
Simplify Sequence diagram and fix class diagram
bdthanh Oct 31, 2022
e6e896b
Merge branch 'branch-update-dg'
bdthanh Oct 31, 2022
ea9ba95
Added Command to find profit of user. Also added extra field in trans…
jorellesee Oct 31, 2022
4892bd2
Fix diagrams
bdthanh Oct 31, 2022
b112276
Reformat HelpCommand
bdthanh Oct 31, 2022
37cf5d9
Fix Glossary
bdthanh Oct 31, 2022
526d042
edited TransactionStorage to accomodate lenderId
jorellesee Oct 31, 2022
e351ca4
Merge pull request #215 from jorellesee/master
bdthanh Oct 31, 2022
af81fb1
Merge branch 'temp-branch' of https://github.com/AY2223S1-CS2113-W12-…
bdthanh Oct 31, 2022
1e95ae0
Fix test and Transaction Storage after updating Transaction lender
bdthanh Oct 31, 2022
30ba2c3
Merge pull request #216 from bdthanh/branch-fix-storage
bdthanh Oct 31, 2022
a30796d
Minor change in UG
bdthanh Oct 31, 2022
7ca71fe
tweaks to DG
chiewyx Oct 31, 2022
02e30ef
fix merge conflicts
chiewyx Oct 31, 2022
b20bbc1
updated user stories
chiewyx Oct 31, 2022
9ae89a8
updated DG to include view user's loss in view user sequence diagram
chiewyx Oct 31, 2022
1402cac
Merge pull request #217 from chiewyx/branch-DG
chiewyx Oct 31, 2022
8d24a68
refactor code based on comments
chiewyx Oct 31, 2022
ce6556f
refactored code based on comments
chiewyx Oct 31, 2022
181daeb
Merge pull request #218 from chiewyx/sort-items
chiewyx Oct 31, 2022
57d9c0a
Handle more exceptions of storage
bdthanh Oct 31, 2022
d4301b2
Merge branch 'master' of https://github.com/AY2223S1-CS2113-W12-1/tp
bdthanh Oct 31, 2022
298218e
Remove magic literals
bdthanh Oct 31, 2022
d47a96a
Add logger
bdthanh Oct 31, 2022
2c7beba
Add javadoc for logger
bdthanh Oct 31, 2022
5814991
Fix magic literals
bdthanh Oct 31, 2022
fd97dda
Merge pull request #220 from bdthanh/branch-logger
bdthanh Oct 31, 2022
01f44ef
Refactor all files with javadocs and fix all checkstyle checks
winston-lim Nov 1, 2022
a3dd1a4
Merge pull request #221 from winston-lim/master
winston-lim Nov 1, 2022
4afbb39
Handle tampered data files
bdthanh Nov 1, 2022
3770d8a
Merge branch 'master' of https://github.com/AY2223S1-CS2113-W12-1/tp …
bdthanh Nov 1, 2022
b2bd98c
Clean code and fix storage
bdthanh Nov 1, 2022
a181992
Fix DG
bdthanh Nov 1, 2022
892096b
Merge pull request #223 from bdthanh/branch-fix-storage-2
bdthanh Nov 1, 2022
05ab055
Merge branch 'master' into master
bdthanh Nov 2, 2022
e40816e
Add architecture
winston-lim Nov 2, 2022
54d17d8
Merge pull request #224 from winston-lim/master
winston-lim Nov 2, 2022
3085cfb
added instructions for manual testing
chiewyx Nov 2, 2022
7f829e6
Merge pull request #225 from chiewyx/branch-DG
chiewyx Nov 2, 2022
1c5aece
Clean main
bdthanh Nov 2, 2022
919d354
Fix checkstyle
bdthanh Nov 2, 2022
bc420ab
Merge pull request #226 from bdthanh/master
bdthanh Nov 2, 2022
d871c97
Catch only DukeException, fix logger
bdthanh Nov 2, 2022
780b634
Merge pull request #227 from bdthanh/master
bdthanh Nov 2, 2022
7e2f37e
edit class diagram
jingwei55 Nov 2, 2022
bba6aa9
Merge branch 'master' of https://github.com/jingwei55/tp
jingwei55 Nov 2, 2022
081dd9a
First fix
bdthanh Nov 3, 2022
7679497
Fix storage and handle more errors
bdthanh Nov 3, 2022
fb6fa7b
Improve storage
bdthanh Nov 3, 2022
cbeecba
Update DG
bdthanh Nov 3, 2022
340a20a
Merge pull request #211 from jingwei55/master
bdthanh Nov 3, 2022
1bffd8d
Merge pull request #228 from bdthanh/branch-fix-storage-3
bdthanh Nov 3, 2022
b5540b9
Update DG
bdthanh Nov 3, 2022
64116a3
Merge pull request #229 from bdthanh/branch-fix-storage-3
bdthanh Nov 3, 2022
d0b7721
Updated UG and DG for user gain/loss and borrow/lend transactions
jorellesee Nov 3, 2022
294b1bf
Merge pull request #230 from jorellesee/master
jorellesee Nov 3, 2022
ece96e0
Update UG DG
bdthanh Nov 4, 2022
29562ea
Merge pull request #232 from bdthanh/master
bdthanh Nov 4, 2022
af5daf2
Update DG
bdthanh Nov 4, 2022
8ad5259
Refactor magic literals
jorellesee Nov 4, 2022
ecd7286
Add PPP for winston
winston-lim Nov 4, 2022
3d19a00
Merge pull request #234 from winston-lim/master
winston-lim Nov 4, 2022
bec7bb0
more refactoring of magic literals
jorellesee Nov 4, 2022
a902407
Merge pull request #235 from jorellesee/master
jorellesee Nov 4, 2022
0f936c7
Fix PPP
winston-lim Nov 4, 2022
a72e482
Merge pull request #236 from winston-lim/master
winston-lim Nov 4, 2022
1c78863
updated PPP
chiewyx Nov 5, 2022
698292e
Merge branch 'master' of https://github.com/AY2223S1-CS2113-W12-1/tp …
chiewyx Nov 5, 2022
8e475fa
Merge pull request #237 from chiewyx/branch-DG
chiewyx Nov 5, 2022
f720ae6
updated PPP and cleaned UG
chiewyx Nov 5, 2022
f1d9c07
Merge pull request #239 from chiewyx/branch-DG
chiewyx Nov 5, 2022
a5d9f7e
no message
jorellesee Nov 5, 2022
74d51b9
Merge pull request #241 from jorellesee/master
jorellesee Nov 5, 2022
2eb1c74
added more tests
chiewyx Nov 5, 2022
6a4bade
Merge branch 'master' of https://github.com/AY2223S1-CS2113-W12-1/tp …
chiewyx Nov 5, 2022
f630fea
Merge pull request #242 from chiewyx/branch-tests
chiewyx Nov 5, 2022
d69a164
Update DG
bdthanh Nov 5, 2022
46d61cb
Merge branch 'master' of https://github.com/AY2223S1-CS2113-W12-1/tp
bdthanh Nov 5, 2022
d35c506
Fix delims in command summary UG
bdthanh Nov 5, 2022
4d4bdd7
Fix bdthanh PPP
bdthanh Nov 5, 2022
b5576c4
Merge pull request #243 from bdthanh/master
bdthanh Nov 5, 2022
21929ef
updated PPP
jorellesee Nov 6, 2022
9d0c75b
Merge pull request #244 from jorellesee/master
jorellesee Nov 6, 2022
c3cf290
Update DG
bdthanh Nov 6, 2022
c8b3279
Update DG
bdthanh Nov 6, 2022
0c7a4ce
Update bdthanh PPP
bdthanh Nov 6, 2022
f07d90d
updated diagrams in DG
jorellesee Nov 6, 2022
f3b4345
Merge pull request #245 from jorellesee/master
jorellesee Nov 6, 2022
d6db7ff
Fix test and checkstyle
bdthanh Nov 6, 2022
f5f13dc
Update DG
bdthanh Nov 6, 2022
846f685
Merge pull request #246 from bdthanh/master
bdthanh Nov 6, 2022
d6dade9
Update PPP and DG
bdthanh Nov 6, 2022
e05de7d
Fix DukeSequence
bdthanh Nov 6, 2022
1b71447
Merge pull request #247 from bdthanh/master
bdthanh Nov 6, 2022
7dc05c6
fix formatting
jingwei55 Nov 6, 2022
79986d8
Increased Tseting for ItemList and Item
jorellesee Nov 6, 2022
c0bc784
Merge pull request #248 from jorellesee/master
jorellesee Nov 6, 2022
5a00d67
Clean code
bdthanh Nov 6, 2022
f448402
standardize keywords
jingwei55 Nov 6, 2022
63f9bd4
Merge branch 'master' of https://github.com/AY2223S1-CS2113-W12-1/tp
bdthanh Nov 6, 2022
09c5c99
Fix checkstyle
bdthanh Nov 6, 2022
1d9a9f6
Update jingwei55.md
jingwei55 Nov 6, 2022
3a28e9d
Merge pull request #249 from bdthanh/master
bdthanh Nov 6, 2022
884d684
Update AboutUs.md
jingwei55 Nov 6, 2022
e544371
fix PPP links
chiewyx Nov 6, 2022
8e7de43
standardise 2dp for price
jingwei55 Nov 6, 2022
5da6f8a
Merge pull request #250 from chiewyx/branch-DG
chiewyx Nov 6, 2022
03c9f77
Standardise 2dp for price in code
bdthanh Nov 6, 2022
2265235
Merge branch 'master' of https://github.com/AY2223S1-CS2113-W12-1/tp
bdthanh Nov 6, 2022
05d71e3
ensure smallest unit of price i1 cent
jingwei55 Nov 6, 2022
a03c95d
added exception messages for too many decimal places in price
jingwei55 Nov 6, 2022
335c5a0
Merge branch 'master' into master
jingwei55 Nov 6, 2022
727a287
solve merge conflict
jingwei55 Nov 6, 2022
e6a4405
update UG for price decimal reminder
jingwei55 Nov 6, 2022
c62cbe6
format nicely
jingwei55 Nov 6, 2022
fce49f7
finishing touches
jingwei55 Nov 6, 2022
27f20ad
for test
jingwei55 Nov 6, 2022
4040031
add days after duration
jingwei55 Nov 6, 2022
530bbda
fix formatting + add FAQ
jingwei55 Nov 6, 2022
51e926c
Update jingwei55.md
jingwei55 Nov 6, 2022
4cde48c
Merge pull request #251 from jingwei55/master
jingwei55 Nov 6, 2022
4b174a7
edited store failure message
jingwei55 Nov 6, 2022
f11706a
settle error
jingwei55 Nov 6, 2022
bcd808c
final
jingwei55 Nov 6, 2022
867ffcf
Merge pull request #252 from jingwei55/master
jingwei55 Nov 6, 2022
1cd7eb8
Finalize code
bdthanh Nov 6, 2022
094bc80
Update UserGuide
bdthanh Nov 6, 2022
3b6f45b
Fix UG and tests
bdthanh Nov 6, 2022
4efb5c3
Merge branch 'master' of https://github.com/AY2223S1-CS2113-W12-1/tp
bdthanh Nov 6, 2022
a82a911
Fix error message and UG
bdthanh Nov 6, 2022
e0b11b5
Merge pull request #253 from bdthanh/master
bdthanh Nov 6, 2022
42a31df
UpdateUG
bdthanh Nov 6, 2022
a44656b
Fix FAQ numbering
bdthanh Nov 6, 2022
7ff88e6
UG
bdthanh Nov 6, 2022
e240ef8
Merge branch 'master' of https://github.com/AY2223S1-CS2113-W12-1/tp
bdthanh Nov 6, 2022
672c225
Merge pull request #254 from bdthanh/master
bdthanh Nov 6, 2022
43ad926
Merge branch 'master' of https://github.com/AY2223S1-CS2113-W12-1/tp
bdthanh Nov 6, 2022
e5a9c86
Fix UG DG
bdthanh Nov 6, 2022
5cc22b4
Fix bugs
bdthanh Nov 6, 2022
bce0879
Improve message
bdthanh Nov 6, 2022
f23792c
RepoSense
bdthanh Nov 6, 2022
0696990
Fix bdthanh image
bdthanh Nov 6, 2022
a784526
Merge pull request #255 from bdthanh/master
bdthanh Nov 6, 2022
176eeda
fix typo
jingwei55 Nov 6, 2022
63a4eee
fix typo
jingwei55 Nov 6, 2022
f566439
Merge pull request #256 from jingwei55/master
jingwei55 Nov 6, 2022
1ef19c2
Polish UG DG
bdthanh Nov 6, 2022
c8478a8
Minor change in DG
bdthanh Nov 7, 2022
9a49f69
Update jorelle.md
jorellesee Nov 7, 2022
d63c18f
uploaded jorelle_image
jorellesee Nov 7, 2022
53ec2c8
Update AboutUs.md
jorellesee Nov 7, 2022
fc01cbe
Rename jorelle.md to jorellesee.md
jorellesee Nov 7, 2022
a793998
Update AboutUs.md
jorellesee Nov 7, 2022
7647aae
Update jingwei55.md
jingwei55 Nov 7, 2022
cd8bbb8
add jw image
jingwei55 Nov 7, 2022
b8aab96
Delete Jing Wei_Chien (2).jpg
jingwei55 Nov 7, 2022
52fafac
Add files via upload
jingwei55 Nov 7, 2022
289fb14
Update AboutUs.md
jingwei55 Nov 7, 2022
d477960
Reformat table (make it 600 commits ^^)
bdthanh Nov 7, 2022
a467abe
Final commit
bdthanh Nov 7, 2022
87f9e3e
Merge pull request #257 from bdthanh/master
bdthanh Nov 7, 2022
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
5 changes: 5 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
# IDEA files
/.idea/
/out/
/data/
/log/
/*.iml

# Gradle build files
Expand All @@ -15,3 +17,6 @@ bin/

/text-ui-test/ACTUAL.txt
text-ui-test/EXPECTED-UNIX.TXT

# Ignore compiled .class files
**/*.class
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# Duke project template
# Duke 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.

Expand Down
3 changes: 2 additions & 1 deletion build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,7 @@ checkstyle {
toolVersion = '8.23'
}

run{
run {
standardInput = System.in
enableAssertions = true
}
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 |
|------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|:--------------:|:----------------------------------------:|:------------------------------:|
| ![Jing Wei_Chien (2)](https://user-images.githubusercontent.com/47519031/200161054-c0ce283c-f5d0-42d8-915c-2bd68b1e2dd8.jpg) | Jing Wei | [Github](https://github.com/jingwei55) | [Portfolio](team/jingwei55.md) |
| ![](https://scontent-xsp1-3.xx.fbcdn.net/v/t39.30808-6/283972742_3147048338875348_1139762696099650067_n.jpg?_nc_cat=107&ccb=1-7&_nc_sid=09cbfe&_nc_ohc=pMTUttIzJhcAX91pn4-&_nc_ht=scontent-xsp1-3.xx&oh=00_AT8wbld3wdGbuhnKgTJPw-OoOjjVNkgrPuCcyAYJfLglcQ&oe=63406F3E) | Bui Duc Thanh | [Github](https://github.com/bdthanh) | [Portfolio](team/bdthanh.md) |
| ![](https://via.placeholder.com/100.png?text=Photo) | Jorelle See | [Github](https://github.com/jorellesee) | [Portfolio](team/jorelle.md) |
| ![](https://i.imgur.com/zeMd6dI.jpg) | Chiew Yi Xiang | [Github](https://github.com/chiewyx) | [Portfolio](team/chiewyx.md) |
| ![winston-profile-photo](https://www.winston-lim.com/profile-avatar.jpeg) | Winston Lim | [Github](https://github.com/winston-lim) | [Portfolio](team/winston.md) |
765 changes: 745 additions & 20 deletions docs/DeveloperGuide.md

Large diffs are not rendered by default.

9 changes: 7 additions & 2 deletions docs/README.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,11 @@
# Duke
# Upcycle

{Give product intro here}
Hi there,

Welcome to Upcycle - a perfect desktop app dedicate to managing rental businesses. The nature of these businesses
requires dealing with tons of data; therefore, noting down all information with notes and pen is not ideal for managers.
It is developed for rental business managers, who can type fast to efficiently keep track of all of their customers, items,
and transactions via a Command Line Interface.

Useful links:
* [User Guide](UserGuide.md)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Activation box ought to not immediately be closed, should leave some time for clearing the object up
1

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello, may I ask why here use <>? Maybe yall trying to express <>
2

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here abt the input, it's a bit confusing for me.
Sine when reading a input, the control alrd should be passed to Ui and later return it.
3

For the second input, it never returns any control

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi this 0 can be omitted~ But it's optional just want u to know
5

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This new User() constructor box should end after the constructor returns
6

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh I think the addUser() func never returns qwq
Uploading 6.png…

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ohhhhh this activation bar is half in the alt part
7

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ohhhhh not return again leh
8

Expand Down
Loading