-
Notifications
You must be signed in to change notification settings - Fork 5
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 tips and warnings #208
Update UG tips and warnings #208
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Check my comments!
docs/UserGuide.md
Outdated
<div markdown="block" class="alert alert-danger"> | ||
|
||
:warning: **Warning:** | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is more of a note instead of a warning
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
haha actl i put under notes, but the doc was under warning but ok i'll change it back
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ohh okok can, warnings are for like irreversible effects, like delete account
docs/UserGuide.md
Outdated
<div markdown="block" class="alert alert-danger"> | ||
|
||
:warning: **Warning:** | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is solved already. Clear command clears only entries filtered.
|
||
:warning: **Warning:** | ||
|
||
* Entry index should only be a single integer, for example, `delete 1 2 c/revenue` will not work. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be a note uh
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
delete shouldn't have an irreversible effect since there is an undo command
No description provided.