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 UserGuide.md #276

Merged
merged 1 commit into from
Nov 10, 2023
Merged

Conversation

andrefoo
Copy link

Add parameters table as a centralised place to view the contraints for each parameter. Resolve #230

Add Cautions for frequently made mistakes. addevent does not update Ui if the event is outside the current week. Resolve #212

Edit fields that are potentially misleadingly name.
Examples:

  • Cca: 2103T Lecture
  • setReminder CS2103T Lecture (cannot add reminder to repeating events)

Add parameters table as a centralised place to view the contraints for each parameter.

Add Cautions for frequently made mistakes. addevent does not update Ui if the event is outside the current week.

Edit fields that are potentially misleadingly name.
Examples:
- Cca: 2103T Lecture
- setReminder CS2103T Lecture (cannot add reminder to repeating events)
@andrefoo andrefoo added documentation Improvements or additions to documentation priority.High Must do labels Nov 10, 2023
@andrefoo andrefoo added this to the v1.4 milestone Nov 10, 2023
@andrefoo andrefoo self-assigned this Nov 10, 2023
Copy link

codecov bot commented Nov 10, 2023

Codecov Report

Merging #276 (58bf846) into master (d9c2f05) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #276   +/-   ##
=========================================
  Coverage     75.59%   75.59%           
  Complexity      949      949           
=========================================
  Files           111      111           
  Lines          3012     3012           
  Branches        351      351           
=========================================
  Hits           2277     2277           
  Misses          649      649           
  Partials         86       86           

Copy link

@kristayeo kristayeo left a comment

Choose a reason for hiding this comment

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

LGTM!

@kristayeo kristayeo merged commit 1e8e969 into AY2324S1-CS2103T-W12-4:master Nov 10, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation priority.High Must do
Projects
None yet
2 participants