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-W13-4] KnottyPlanners #59

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

Conversation

jonushzw
Copy link

KnottyPlanners helps wedding planners detail their clients and contractors. It is optimised for CLI users to be done in commands

Copy link

codecov bot commented Oct 2, 2024

Codecov Report

Attention: Patch coverage is 82.29508% with 54 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/main/java/seedu/address/ui/MainWindow.java 0.00% 30 Missing ⚠️
...rc/main/java/seedu/address/model/ModelManager.java 88.05% 7 Missing and 1 partial ⚠️
src/main/java/seedu/address/ui/ResultDisplay.java 0.00% 5 Missing ⚠️
...eedu/address/logic/commands/AddWeddingCommand.java 81.81% 0 Missing and 4 partials ⚠️
src/main/java/seedu/address/MainApp.java 0.00% 2 Missing ⚠️
...eedu/address/logic/parser/FilterCommandParser.java 86.66% 0 Missing and 2 partials ⚠️
...in/java/seedu/address/logic/parser/ParserUtil.java 93.10% 1 Missing and 1 partial ⚠️
...va/seedu/address/logic/commands/TagAddCommand.java 88.88% 1 Missing ⚠️
Files with missing lines Coverage Δ Complexity Δ
...in/java/seedu/address/commons/util/StringUtil.java 96.29% <100.00%> (+1.55%) 12.00 <5.00> (+5.00)
src/main/java/seedu/address/logic/Messages.java 76.74% <100.00%> (+6.15%) 6.00 <0.00> (ø)
.../java/seedu/address/logic/commands/AddCommand.java 82.25% <100.00%> (+14.98%) 19.00 <0.00> (+6.00)
...ddress/logic/commands/ClearAddressBookCommand.java 100.00% <ø> (ø) 2.00 <0.00> (ø)
...ddress/logic/commands/ClearWeddingBookCommand.java 100.00% <ø> (ø) 2.00 <0.00> (ø)
...va/seedu/address/logic/commands/DeleteCommand.java 100.00% <ø> (ø) 9.00 <0.00> (+1.00)
...a/seedu/address/logic/commands/DeleteNCommand.java 100.00% <ø> (ø) 3.00 <0.00> (ø)
...u/address/logic/commands/DeleteWeddingCommand.java 100.00% <ø> (ø) 8.00 <0.00> (ø)
...a/seedu/address/logic/commands/DeleteYCommand.java 100.00% <100.00%> (ø) 12.00 <0.00> (-3.00)
...java/seedu/address/logic/commands/EditCommand.java 97.61% <100.00%> (+0.05%) 16.00 <0.00> (ø)
... and 35 more

... and 2 files with indirect coverage changes

potaotototo pushed a commit to potaotototo/tp that referenced this pull request Oct 16, 2024
…able-values

Modify `Name` to reflect acceptable values in MVP document
zzibo and others added 28 commits November 3, 2024 15:57
trims the names of bride and groom before adding and capitalise them
Update UG to reflect new command format
…Message

Update command word in wedding doesnt exist
Update some screenshots also
Update some screenshots also
jonushzw and others added 30 commits November 7, 2024 20:25
bug fix: Add in methods to allow clear-ab and clear-wb to remove wedding participants and wedding tags respectively
…tmessage

Add new line character before "Parameters"
Update CodeCov and CI links in readme and index.md
Sample data was wrong and final checkstyle errors
update button references correctly
bug fix: Add error message when executing deleteNCommand with no pending operation
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