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-1] Networkers #40

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

Conversation

robinrojh
Copy link

Networkers helps ISP technicians who visit their clients often remember the details of their clients and the procedures done during the visit to the client. It is optimized for CLI users, assisting them in storing important information quickly.

@codecov-commenter
Copy link

codecov-commenter commented Feb 13, 2022

Codecov Report

Merging #40 (23caa66) into master (4f7660a) will decrease coverage by 0.38%.
The diff coverage is 73.90%.

@@             Coverage Diff              @@
##             master      #40      +/-   ##
============================================
- Coverage     72.15%   71.77%   -0.39%     
- Complexity      399      595     +196     
============================================
  Files            70      100      +30     
  Lines          1232     2069     +837     
  Branches        125      234     +109     
============================================
+ Hits            889     1485     +596     
- Misses          311      478     +167     
- Partials         32      106      +74     
Impacted Files Coverage Δ
...n/java/seedu/address/commons/core/GuiSettings.java 69.23% <ø> (ø)
...main/java/seedu/address/commons/core/Messages.java 0.00% <ø> (ø)
.../seedu/address/logic/parser/FindCommandParser.java 100.00% <ø> (ø)
.../main/java/seedu/address/model/client/Address.java 80.00% <ø> (ø)
...rc/main/java/seedu/address/model/client/Email.java 80.00% <ø> (ø)
src/main/java/seedu/address/model/client/Name.java 80.00% <ø> (ø)
...rc/main/java/seedu/address/model/client/Phone.java 80.00% <ø> (ø)
.../java/seedu/address/model/util/SampleDataUtil.java 5.88% <0.00%> (-14.12%) ⬇️
src/main/java/seedu/address/ui/ClientCard.java 0.00% <0.00%> (ø)
...rc/main/java/seedu/address/ui/ClientListPanel.java 0.00% <0.00%> (ø)
... and 61 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4f7660a...23caa66. Read the comment docs.

robinrojh and others added 29 commits April 7, 2022 16:35
Improve consistency of the UG
Correct ClearCommand's behavior when reflecting cleared Procedures
Add contributions to JB's PPP
Edit listProc and list GUI images on UG
This PR updates Alexander's portion of the PPP as specified by tP guidelines.
iztanpy and others added 30 commits April 11, 2022 12:57
Current Developer guide is not updated.

This change is important to allow better
readability from future SWEs.

Edit Developer Guide
Fix FindCommand functionality bug
The current standard does not reflect
the program.

This is important to increase readability for our
users.

Edit UserGuide.md.
…input

Update cost to reject incorrect input
Edit User Guide and Model Diagram for Procedure
The current Developer Guide is not updated.

This is important to reflect the newly
updated Developer Guide.

Edit Developer Guide.
Update architecture sequence diagram
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.

6 participants