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 UG for Complete and Schedule Command to be clearer #219

Merged
merged 2 commits into from
Nov 7, 2023

Conversation

Kb-Tay
Copy link
Collaborator

@Kb-Tay Kb-Tay commented Nov 7, 2023

Previously, the format for user's date input for Complete and Schedule is unclear

UG is update to clearly differentiate the input for Complete and Schedule command. The way complete command works is also illustrated more clearly

This is to prevent confusion when using the features

Kb-Tay and others added 2 commits November 7, 2023 17:13
Previously, the format for user's date input for Complete and Schedule is unclear

UG is update to clearly differentiate the input for Complete and Schedule command. The way complete command works is also illustrated more clearly

This is to prevent confusion when using the features
Copy link

codecov bot commented Nov 7, 2023

Codecov Report

Merging #219 (0aa9aaa) into master (2dc919f) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #219   +/-   ##
=========================================
  Coverage     75.82%   75.82%           
  Complexity      671      671           
=========================================
  Files           103      103           
  Lines          2147     2147           
  Branches        222      222           
=========================================
  Hits           1628     1628           
  Misses          459      459           
  Partials         60       60           

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

Copy link
Collaborator

@jylow jylow left a comment

Choose a reason for hiding this comment

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

LGTM

@jylow jylow merged commit 2aae7e7 into AY2324S1-CS2103T-F12-1:master Nov 7, 2023
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.

2 participants