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

[CS2103T-F10-1] ClientHub #124

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

Conversation

rubinnn
Copy link

@rubinnn rubinnn commented Sep 21, 2024

Our product provides independent financial advisors with a streamlined tool to manage client details (eg. Track insurance policies). Optimized for simplicity and efficiency, this product makes the lives of financial advisors easier by offering easier access to relevant information for their clients.

Copy link

codecov bot commented Sep 21, 2024

Codecov Report

Attention: Patch coverage is 80.27444% with 115 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/main/java/seedu/address/ui/HelpWindow.java 0.00% 25 Missing ⚠️
...c/main/java/seedu/address/ui/ViewPersonWindow.java 0.00% 24 Missing ⚠️
src/main/java/seedu/address/MainApp.java 0.00% 12 Missing ⚠️
.../java/seedu/address/model/util/SampleDataUtil.java 10.00% 9 Missing ⚠️
src/main/java/seedu/address/ui/MainWindow.java 0.00% 9 Missing ⚠️
...ava/seedu/address/model/person/NameComparator.java 45.45% 2 Missing and 4 partials ⚠️
...ava/seedu/address/model/clienttype/ClientType.java 71.42% 2 Missing and 2 partials ⚠️
src/main/java/seedu/address/ui/PersonCard.java 0.00% 4 Missing ⚠️
...java/seedu/address/logic/commands/ViewCommand.java 87.50% 2 Missing and 1 partial ⚠️
...u/address/logic/parser/FindPhoneCommandParser.java 70.00% 2 Missing and 1 partial ⚠️
... and 11 more
Files with missing lines Coverage Δ Complexity Δ
...in/java/seedu/address/commons/core/LogsCenter.java 78.78% <ø> (ø) 5.00 <0.00> (ø)
src/main/java/seedu/address/logic/Messages.java 88.88% <100.00%> (+1.38%) 3.00 <0.00> (ø)
.../java/seedu/address/logic/commands/AddCommand.java 100.00% <ø> (ø) 8.00 <0.00> (ø)
...ava/seedu/address/logic/commands/ClearCommand.java 100.00% <100.00%> (ø) 2.00 <0.00> (ø)
...va/seedu/address/logic/commands/CommandResult.java 100.00% <100.00%> (ø) 15.00 <4.00> (+2.00)
...va/seedu/address/logic/commands/DeleteCommand.java 100.00% <100.00%> (ø) 8.00 <1.00> (+1.00)
...java/seedu/address/logic/commands/ExitCommand.java 100.00% <100.00%> (ø) 2.00 <1.00> (ø)
...edu/address/logic/commands/FindAddressCommand.java 100.00% <100.00%> (ø) 6.00 <6.00> (?)
.../address/logic/commands/FindClientTypeCommand.java 100.00% <100.00%> (ø) 6.00 <6.00> (?)
...java/seedu/address/logic/commands/FindCommand.java 100.00% <100.00%> (ø) 6.00 <0.00> (ø)
... and 46 more

ahmadsyuaib and others added 29 commits October 23, 2024 15:25
Update user guide to include Sort Command
Change the UI to make it slightly more appealing
Add some ViewPersonWindow CSS to make it look more appealing
Update UI for MainWindow and ViewPersonWindow
Add more test cases for FindCommandParserTest.java
Update developer guide regarding model
Update DeleteSequenceDiagram
Update UndoSequenceDiagram-Logic
Update UndoSequenceDiagram-Model
Update FindCommand and FindNameCommand in the UG
Update the invalid input test case
Add new test cases for invalid inputs
Update user guide and developer guide
Update the UML diagram that was illustrating the UI components
Update the links to the relevant files under the UI and Logic section
Replace "addressbook" with clienthub under the Logic section for some
text.
Update the AddCommand in UG
Refine the formatting for FindCommand. Remove the redundant description
under the shortcut find commands.
Update description to now be able to take
up to 500 characters up from 150. Update
clientType to be compulsory
Update description and clientType
Refine the feature section in the Userguide
L-rrrr and others added 27 commits November 6, 2024 20:06
Add view in the use case
Modify the ViewCommandParser
Update the glossary
Make the find name predicate more succinct
Modify the algorithm for the different finds.
Update DG with new instructions
as well as new BetterModelClassDiagram
Updated Branch changes to fix
merge conflicts
Before the fix, when users view a person and close the app, the view
window remains open. Now, when the user close the app without closing
the view window, the viewperson window will close as well.
Update Developer Guide
with correct relevent information
Update DG to adhere to style
Update UI image to reflect latest version
Add more guide on a valid email and address
Update quickstart of User guide
to be more beginner friendly
Removed the "T" showing between the date and time for output messages of
reminder commands.
Fix the date time format for reminder commands
juliantayyc pushed a commit to juliantayyc/tp that referenced this pull request Nov 7, 2024
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.

5 participants