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

Add user guide for tags. #21

Merged
merged 13 commits into from
Sep 27, 2020

Conversation

Asthenosphere
Copy link

#3

  • Add user guide for the tags section.

@Asthenosphere Asthenosphere added this to the v1.1 milestone Sep 27, 2020
Copy link

@sebastiantoh sebastiantoh left a comment

Choose a reason for hiding this comment

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

Looks good!
P.S. I have some comments on stuff which I didn't notice earlier when we were doing up the UG draft

#### Editing a tag: `tag edit`

Edits an existing tag in the StonkBook to the specified tag name. All entries previously associated with this tag will be updated to associated with the updated tag.

Choose a reason for hiding this comment

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

Suggested change
Edits an existing tag in the StonkBook to the specified tag name. All entries previously associated with this tag will be updated to associated with the updated tag.
Edits an existing tag in StonksBook to the specified tag name. All entries previously associated with this tag will be updated to associated with the updated tag.

I think we can remove the here

* Edits the name of the tag at the specified `INDEX` to be the specified `NAME`. The `INDEX` refers to the index number shown in the list displayed by the `tag list` command.
* The `INDEX` must be a positive integer 1, 2, 3, ...
* The `NAME` and `INDEX` fields must be provided.
* All contacts that have been previously associated with this tag will be updated automatically to associate with the updated tag.

Choose a reason for hiding this comment

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

Suggested change
* All contacts that have been previously associated with this tag will be updated automatically to associate with the updated tag.
* All contacts that have been previously associated with this tag will be updated automatically to be associated with the updated tag.


Examples:

* `tag edit 1 n/close friends` updates the tag `friends` to `close friends`.

Choose a reason for hiding this comment

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

Should we mention friends here? I think it may be better if you just mention that it updates the name of the first tag (as per tag list)


Examples:

* `tag delete 1` deletes the tag `friends` from the tag list.

Choose a reason for hiding this comment

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

Same comment for here (about whether to mention friends)


Format: `tag find INDEX [MODEL]`

* Displays all entries of `[MODEL]` (optional, see point 3 below) associated with the tag at `INDEX`. The `INDEX` refers to the index number shown in the list displayed by the `tag list` command and `[MODEL]` refers to the model to search for (i.e. contacts, sales, etc.).

Choose a reason for hiding this comment

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

Maybe we should consider switching to an ordered list if we're going to make reference to other points

@sebastiantoh sebastiantoh merged commit 44c22e7 into AY2021S1-CS2103T-T11-1:master Sep 27, 2020
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.

None yet

2 participants