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

Update user guide #270

Merged
merged 1 commit into from
Nov 11, 2023
Merged

Conversation

jylow
Copy link
Collaborator

@jylow jylow commented Nov 11, 2023

Several minor bugs were identified with error message displaying incorrect information and UserGuide containing misleading information.

Bugs have been rectified and additional information added to the User Guide.

Fixes #237, #243, #264, #265, #267

Several minor bugs were identified with error message displaying
incorrect information and UserGuide containing misleading information.

Bugs have been rectified and additional information added to the
User Guide.
Copy link

codecov bot commented Nov 11, 2023

Codecov Report

Merging #270 (10d5d1e) into master (1477bcd) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #270   +/-   ##
=========================================
  Coverage     75.58%   75.58%           
  Complexity      680      680           
=========================================
  Files           103      103           
  Lines          2183     2183           
  Branches        223      223           
=========================================
  Hits           1650     1650           
  Misses          469      469           
  Partials         64       64           
Files Coverage Δ
.../seedu/address/logic/commands/CompleteCommand.java 100.00% <ø> (ø)

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

Copy link
Collaborator

@sopa301 sopa301 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sopa301 sopa301 merged commit 141061e into AY2324S1-CS2103T-F12-1:master Nov 11, 2023
5 checks passed
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.

Update DG future implementation of checking duplicate contacts
2 participants