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

[CS2103-W14-1] VendorVault #139

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

Conversation

prave1n
Copy link

@prave1n prave1n commented Sep 21, 2024

VendorVault is a supplier and delivery management app that allows grocers to efficiently add, delete, and mark suppliers and deliveries while tracking their completion status

Copy link

codecov bot commented Sep 21, 2024

Codecov Report

Attention: Patch coverage is 75.39683% with 62 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
.../seedu/address/logic/parser/AddressBookParser.java 72.09% 7 Missing and 5 partials ⚠️
src/main/java/seedu/address/ui/SupplierCard.java 0.00% 10 Missing ⚠️
...in/java/seedu/address/model/delivery/DateTime.java 61.11% 5 Missing and 2 partials ⚠️
...du/address/logic/commands/MarkDeliveryCommand.java 78.94% 0 Missing and 4 partials ⚠️
.../address/logic/commands/DeleteDeliveryCommand.java 66.66% 0 Missing and 3 partials ⚠️
...ess/model/delivery/DeliverySortCostComparator.java 57.14% 0 Missing and 3 partials ⚠️
...model/delivery/DeliverySortDateTimeComparator.java 57.14% 0 Missing and 3 partials ⚠️
...s/model/delivery/DeliverySortStatusComparator.java 50.00% 0 Missing and 3 partials ⚠️
...edu/address/logic/commands/AddDeliveryCommand.java 66.66% 0 Missing and 2 partials ⚠️
...in/java/seedu/address/logic/parser/ParserUtil.java 66.66% 1 Missing and 1 partial ⚠️
... and 10 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 96.72% <100.00%> (+9.22%) 8.00 <1.00> (+5.00)
.../java/seedu/address/logic/commands/AddCommand.java 100.00% <100.00%> (ø) 1.00 <1.00> (-7.00)
...edu/address/logic/commands/AddSupplierCommand.java 100.00% <ø> (ø) 8.00 <0.00> (?)
...va/seedu/address/logic/commands/DeleteCommand.java 100.00% <100.00%> (ø) 1.00 <1.00> (-6.00)
.../address/logic/commands/DeleteSupplierCommand.java 100.00% <100.00%> (ø) 7.00 <1.00> (?)
...java/seedu/address/logic/commands/ExitCommand.java 100.00% <ø> (ø) 2.00 <0.00> (ø)
...du/address/logic/commands/FindSupplierCommand.java 90.47% <ø> (ø) 8.00 <0.00> (?)
...a/seedu/address/logic/commands/ListAllCommand.java 100.00% <100.00%> (ø) 2.00 <1.00> (?)
...java/seedu/address/logic/commands/ListCommand.java 100.00% <100.00%> (ø) 1.00 <1.00> (-1.00)
... and 70 more

... and 1 file with indirect coverage changes

CJianzhi and others added 29 commits November 7, 2024 19:18
…isplay

Standardise delete commands to display all entries
Update Architecture sequence diagram and NFRs in DG
Include AddCommand class to encapsulate AddSupplierCommand and AddDeliveryCommand
Make DeleteCommand an abstract class that encapsulates DeleteSupplierCommand and DeleteDeliveryCommand
Update MarkSupplierCommand and MarkDeliveryCommand to inherit from MarkCommand
Include JavaDocs for SortSupplierCommand and a few other constructors
Improve parsing logic in AddressBookParser
Remove Address Class
…eview-branch

# Conflicts:
#	src/main/java/seedu/address/logic/commands/DeleteDeliveryCommand.java
#	src/main/java/seedu/address/logic/commands/DeleteSupplierCommand.java
Update user stories to developer guide
Update logic component in developer guide
…eview-branch

# Conflicts:
#	docs/DeveloperGuide.md
#	docs/UserGuide.md
#	src/main/java/seedu/address/logic/parser/AddressBookParser.java
Update delivery guide for delivery test instructions
Show duplicate delivery message if cost value is the same
prave1n and others added 30 commits November 12, 2024 12:39
Update JavaDoc comments and improve code quality
Standardise format in DeveloperGuide
Remove spacing for model diagram in DG
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