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-T14-3] MyCRM #87

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

Conversation

HangZelin
Copy link

MyCRM is a desktop application for managing client contacts, repair job statuses, and product information that has been optimised for use via a Command Line Interface (CLI) while maintaining the benefits of a Graphical User Interface (GUI).MyCRM can complete customer relationship management tasks faster than traditional GUI applications.

@codecov-commenter
Copy link

codecov-commenter commented Sep 15, 2021

Codecov Report

Merging #87 (3192fd2) into master (ba53b8c) will decrease coverage by 5.78%.
The diff coverage is 75.02%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #87      +/-   ##
============================================
- Coverage     72.15%   66.37%   -5.79%     
- Complexity      399     1187     +788     
============================================
  Files            70      173     +103     
  Lines          1232     4018    +2786     
  Branches        125      495     +370     
============================================
+ Hits            889     2667    +1778     
- Misses          311     1165     +854     
- Partials         32      186     +154     
Impacted Files Coverage Δ
src/main/java/seedu/mycrm/AppParameters.java 75.00% <ø> (ø)
src/main/java/seedu/mycrm/Main.java 0.00% <ø> (ø)
src/main/java/seedu/mycrm/MainApp.java 0.00% <0.00%> (ø)
src/main/java/seedu/mycrm/commons/core/Config.java 68.18% <ø> (ø)
...main/java/seedu/mycrm/commons/core/LogsCenter.java 78.37% <ø> (ø)
...c/main/java/seedu/mycrm/commons/core/Messages.java 0.00% <0.00%> (ø)
...rc/main/java/seedu/mycrm/commons/core/Version.java 82.50% <ø> (ø)
...ain/java/seedu/mycrm/commons/core/index/Index.java 100.00% <ø> (ø)
...rm/commons/exceptions/DataConversionException.java 100.00% <ø> (ø)
...ycrm/commons/exceptions/IllegalValueException.java 100.00% <ø> (ø)
... and 177 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 ba53b8c...3192fd2. Read the comment docs.

bentanjunrong pushed a commit to bentanjunrong/tp that referenced this pull request Oct 18, 2021
Domszy pushed a commit to Domszy/tp that referenced this pull request Oct 19, 2021
HangZelin and others added 26 commits October 29, 2021 11:55
Add one more exception catch for edit command mutistage
Fix editContactCommand Bug.
Fix the issue of unclearness of `addContact` optional rationale.
To protect clients' pravicy, we allow a client to provide only partial field of his info.
But `n/CLIENT_NAME`  and at least one field of `c/CONTACT_NUMBER`, `e/EMAIL` or `a/ADDRESS` should exist.
Update DG EditTemplate and Fix UG Template
dhshah1 and others added 30 commits November 8, 2021 21:37
Fix description for undoCompleteJobCommand
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