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 documentations for DG, UG and PPP #290

Merged
merged 4 commits into from
Nov 12, 2023

Conversation

AlyssaPng
Copy link
Collaborator

@AlyssaPng AlyssaPng commented Nov 12, 2023

Update DG, UG and PPP

Copy link

codecov bot commented Nov 12, 2023

Codecov Report

Merging #290 (afe01a6) into master (fba5210) will increase coverage by 0.65%.
Report is 8 commits behind head on master.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #290      +/-   ##
============================================
+ Coverage     75.53%   76.19%   +0.65%     
- Complexity      686      694       +8     
============================================
  Files           103      103              
  Lines          2195     2197       +2     
  Branches        223      223              
============================================
+ Hits           1658     1674      +16     
+ Misses          473      467       -6     
+ Partials         64       56       -8     
Files Coverage Δ
...va/seedu/address/logic/commands/GatherCommand.java 100.00% <ø> (ø)
...eedu/address/logic/parser/GatherCommandParser.java 86.36% <ø> (ø)
...in/java/seedu/address/logic/parser/ParserUtil.java 94.00% <ø> (ø)
...edu/address/model/financialplan/FinancialPlan.java 73.33% <ø> (ø)
...c/main/java/seedu/address/model/person/Person.java 93.24% <ø> (ø)
...person/gatheremail/GatherEmailByFinancialPlan.java 100.00% <ø> (ø)
...ess/model/person/gatheremail/GatherEmailByTag.java 100.00% <ø> (ø)
src/main/java/seedu/address/model/tag/Tag.java 73.33% <ø> (ø)

... and 6 files with indirect coverage changes

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

Copy link
Collaborator

@Kb-Tay Kb-Tay left a comment

Choose a reason for hiding this comment

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

LGTM

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

@@ -448,6 +455,8 @@ UNOFAS data are saved automatically as a JSON file `[JAR file location]/data/add

<div markdown="span" class="alert alert-warning"> :exclamation: **Caution:**
If your changes to the data file makes its format invalid, UNOFAS will discard all data and start with an empty data file at the next run. Hence, it is recommended to take a backup of the file before editing it.

Changing the `appointment` date to an invalid month (e.g. 13-13-2023) will cause an error and UNOFAS will not run. However, changing it to an invalid day (e.g. 31-02-2023), UNOFAS will update this date to the closest correct date (e.g. 28-02-2023).
Copy link
Collaborator

Choose a reason for hiding this comment

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

We don't need to state this explicitly, we can just say that only correct edits will work, and that any wrong edits will lead to undefined behaviour, including the deletion of the entire data file.
image

Suggested change
Changing the `appointment` date to an invalid month (e.g. 13-13-2023) will cause an error and UNOFAS will not run. However, changing it to an invalid day (e.g. 31-02-2023), UNOFAS will update this date to the closest correct date (e.g. 28-02-2023).
If your changes to the data file makes its format invalid, UNOFAS will discard all data and start with an empty data file at the next run. Certain edits to the file may also cause unexpected behaviours. Please only edit the file if you are confident that it is correct, and it is recommended to take a backup of the file before editing it.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

okayy will edit

@sopa301 sopa301 merged commit d32d8da into AY2324S1-CS2103T-F12-1:master Nov 12, 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
Projects
None yet
3 participants