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-T13-4] SweetRewards #136

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

Conversation

kokerinks
Copy link

SweetRewards solves the challenge of tracking and rewarding customer loyalty for small bakeries. It helps to provide fast and convenient access to the number of points each member has to provide them with the corresponding benefits, enhancing customer satisfaction and retention.

Copy link

codecov bot commented Mar 5, 2024

Codecov Report

Attention: Patch coverage is 88.34019% with 85 lines in your changes are missing coverage. Please review.

Project coverage is 83.13%. Comparing base (b896eb0) to head (1ff188a).

❗ Current head 1ff188a differs from pull request most recent head c65dea2. Consider uploading reports for the commit c65dea2 to get more accurate results

Files Patch % Lines
.../java/seedu/address/model/person/orders/Order.java 58.62% 6 Missing and 6 partials ⚠️
.../address/logic/parser/DeleteItemCommandParser.java 0.00% 7 Missing ⚠️
...du/address/logic/commands/AddMemPointsCommand.java 81.48% 2 Missing and 3 partials ⚠️
...a/seedu/address/model/person/MembershipPoints.java 83.33% 3 Missing and 2 partials ⚠️
.../seedu/address/logic/commands/AddOrderCommand.java 90.24% 2 Missing and 2 partials ⚠️
.../seedu/address/logic/parser/AddressBookParser.java 55.55% 4 Missing ⚠️
...in/java/seedu/address/logic/parser/ParserUtil.java 87.50% 2 Missing and 2 partials ⚠️
...in/java/seedu/address/model/allergen/Allergen.java 71.42% 2 Missing and 2 partials ⚠️
.../main/java/seedu/address/model/item/Catalogue.java 91.66% 2 Missing and 2 partials ⚠️
...c/main/java/seedu/address/model/person/Person.java 86.20% 1 Missing and 3 partials ⚠️
... and 19 more
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #136      +/-   ##
============================================
+ Coverage     75.26%   83.13%   +7.86%     
- Complexity      419      641     +222     
============================================
  Files            71       90      +19     
  Lines          1338     1891     +553     
  Branches        126      216      +90     
============================================
+ Hits           1007     1572     +565     
+ Misses          301      255      -46     
- Partials         30       64      +34     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

billyhoce and others added 29 commits March 15, 2024 23:32
Fix JUnit tests for AddMemshipCommandParser
Update UG for SeedData, Clear commands
Update Clear command to use --force tag
kinjalagarwal1810 and others added 30 commits April 15, 2024 12:14
Fix bug arising from numbers in item
Fix naming error for seeddata in readme and landing page
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