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-F11-4] BookFace #81

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

Conversation

SHni99
Copy link

@SHni99 SHni99 commented Feb 17, 2023

Students often take many classes and meet different people. Thus, there will be many things on hand that they need to keep track of. With this phenomenon that they faced every day in life, a solution is needed to tackle the issue; hence Bookface. Bookface helps them to organise their contacts list for an easier way to set up proper communication channels. This makes it easier for students to form connections with their peers during their time in university.

@codecov
Copy link

codecov bot commented Feb 17, 2023

Codecov Report

Patch coverage: 52.35% and project coverage change: -7.97 ⚠️

Comparison is base (928f146) 72.15% compared to head (3a9647c) 64.19%.

Additional details and impacted files
@@             Coverage Diff              @@
##             master      #81      +/-   ##
============================================
- Coverage     72.15%   64.19%   -7.97%     
- Complexity      399      480      +81     
============================================
  Files            70       88      +18     
  Lines          1232     1751     +519     
  Branches        125      206      +81     
============================================
+ Hits            889     1124     +235     
- Misses          311      548     +237     
- Partials         32       79      +47     
Impacted Files Coverage Δ
src/main/java/seedu/address/Main.java 0.00% <ø> (ø)
src/main/java/seedu/address/MainApp.java 0.00% <0.00%> (ø)
...n/java/seedu/address/commons/core/GuiSettings.java 69.23% <ø> (ø)
...main/java/seedu/address/commons/core/Messages.java 0.00% <ø> (ø)
...rc/main/java/seedu/address/logic/LogicManager.java 65.21% <0.00%> (-9.79%) ⬇️
...ain/java/seedu/address/logic/commands/Command.java 100.00% <ø> (ø)
...va/seedu/address/logic/commands/DeleteCommand.java 100.00% <ø> (ø)
.../seedu/address/logic/parser/AddressBookParser.java 78.94% <0.00%> (-21.06%) ⬇️
src/main/java/seedu/address/model/AddressBook.java 66.66% <0.00%> (-25.00%) ⬇️
src/main/java/seedu/address/model/Model.java 100.00% <ø> (ø)
... and 50 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

SHni99 and others added 29 commits March 15, 2023 19:28
v1.2 Edit Person toString method to not show image
V1.2 [Fix add-image list filter]
V1.2b [Add status variable in Person class]
More specific error messages for Import command
[v1.2b] Expand tag pane in personal pane
Fix Add-Image does not work on jar
SHni99 and others added 30 commits April 7, 2023 12:01
v1.4 [Update Ui diagram in UG]
Reduce margin size for headers
Slight documentation change
Added testing for import feature
Update ppp for shni99.md
Fix square brackets UG Import
Fix integer overflow bug
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.

4 participants