Skip to content

[CS2103T-F14b-4] F.A.R.T#202

Open
fymbc wants to merge 440 commits into
nus-cs2103-AY2425S1:masterfrom
AY2425S1-CS2103T-F14b-4:master
Open

[CS2103T-F14b-4] F.A.R.T#202
fymbc wants to merge 440 commits into
nus-cs2103-AY2425S1:masterfrom
AY2425S1-CS2103T-F14b-4:master

Conversation

@fymbc

@fymbc fymbc commented Sep 30, 2024

Copy link
Copy Markdown

Financial Advisors Revolutionary Telemarketer (F.A.R.T), is an address book for Financial Advisors who engage frequently or specialise primarily in client outreach.

@codecov

codecov Bot commented Sep 30, 2024

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 51.43443% with 237 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...main/java/seedu/address/ui/PersonDetailedView.java 0.00% 32 Missing ⚠️
...va/seedu/address/logic/commands/UploadCommand.java 28.57% 29 Missing and 1 partial ⚠️
...u/address/ui/PersonDetailedViewContentManager.java 46.93% 25 Missing and 1 partial ⚠️
src/main/java/seedu/address/MainApp.java 0.00% 21 Missing ⚠️
src/main/java/seedu/address/ui/PersonCard.java 0.00% 19 Missing ⚠️
...main/java/seedu/address/ui/ConfirmClearWindow.java 0.00% 18 Missing ⚠️
.../java/seedu/address/ui/DuplicateWarningWindow.java 0.00% 18 Missing ⚠️
src/main/java/seedu/address/ui/MainWindow.java 0.00% 14 Missing ⚠️
.../seedu/address/logic/parser/FindCommandParser.java 45.45% 8 Missing and 4 partials ⚠️
...main/java/seedu/address/model/person/Birthday.java 42.85% 7 Missing and 1 partial ⚠️
... and 15 more
Files with missing lines Coverage Δ Complexity Δ
...n/java/seedu/address/commons/core/GuiSettings.java 76.92% <ø> (ø) 8.00 <0.00> (ø)
src/main/java/seedu/address/logic/Messages.java 90.90% <ø> (ø) 4.00 <0.00> (ø)
...java/seedu/address/logic/commands/FindCommand.java 100.00% <ø> (ø) 6.00 <0.00> (ø)
...a/seedu/address/logic/parser/AddCommandParser.java 100.00% <100.00%> (ø) 5.00 <0.00> (ø)
.../seedu/address/logic/parser/AddressBookParser.java 100.00% <100.00%> (ø) 15.00 <0.00> (+1.00)
...eedu/address/logic/parser/DeleteCommandParser.java 100.00% <100.00%> (ø) 2.00 <0.00> (ø)
.../seedu/address/logic/parser/EditCommandParser.java 93.54% <100.00%> (+0.44%) 13.00 <0.00> (+1.00)
.../seedu/address/logic/parser/PaidCommandParser.java 91.30% <100.00%> (+2.41%) 3.00 <0.00> (ø)
...eedu/address/logic/parser/UnpaidCommandParser.java 100.00% <100.00%> (ø) 2.00 <0.00> (ø)
...eedu/address/logic/parser/UploadCommandParser.java 100.00% <100.00%> (ø) 2.00 <2.00> (?)
... and 34 more

... and 4 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

potaotototo pushed a commit to potaotototo/tp that referenced this pull request Nov 5, 2024
…factor-ui

Refactor UI utility methods into a centralized UiUtil class
Justin-Aw46 and others added 30 commits November 11, 2024 22:55
Revert changes to delete and update DG
Revert changes violating feature freeze
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