Skip to content

[CS2103T-T08-2] OnlySales#141

Open
pohanson wants to merge 445 commits into
nus-cs2103-AY2526S1:masterfrom
AY2526S1-CS2103T-T08-2:master
Open

[CS2103T-T08-2] OnlySales#141
pohanson wants to merge 445 commits into
nus-cs2103-AY2526S1:masterfrom
AY2526S1-CS2103T-T08-2:master

Conversation

@pohanson

Copy link
Copy Markdown

OnlySales helps salesperson to unlock their full potential in managing their customers' contact information by doing everything from the keyboard.

seanrh34 and others added 29 commits October 23, 2025 21:42
Update template to allow copying a template to clipboard
* Remove similar methods `parse` and `readAddressBookFromJsonString`
* Remove nesting of `if (!isNullOrEmptyString(jsonString))`
* Separation of Concern, `JsonAddressBookStorage` deals with storing json file on disk, while `JsonAddressBookUtil` deal with other operations, such as `readingAddressBookFromJsonString`
Update UML diagram for Model component
Update logic and parser class diagrams
* Add documentation
* DRY by using StringUtil
jooh6969 pushed a commit to jooh6969/tp that referenced this pull request Nov 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants