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

UG: Add change command and update remarks #121

Merged
merged 5 commits into from
Oct 25, 2020

Conversation

Avalionnet
Copy link
Collaborator

  • Add change command implementation details in the user guide.
  • Update remark command usage details.

With this update, users will now be able to learn the syntax for rescheduling and modifying appointments.

@Avalionnet Avalionnet added the type.Documentation 📝 Updates to documentation label Oct 24, 2020
@Avalionnet Avalionnet added this to the v1.3b milestone Oct 24, 2020
@Avalionnet Avalionnet self-assigned this Oct 24, 2020
Copy link
Collaborator

@YangYue128-helen YangYue128-helen 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

@JinHao-L JinHao-L left a comment

Choose a reason for hiding this comment

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

LGTM! Just have to handle the merge conflict 👍

<br>
* Creates and **adds a remark** for the patient at the specified `INDEX`. The index here refers to the index number shown on the left side of the displayed patient list.
* Please note that the index used **must be a positive integer** 1, 2, 3, …​
* A remark serves as an **optional field** for adding extra info to a patient's bio data and can be left blank if it is not applicable. modified using the edit/remark command after it has been created.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think the second line can be omitted. (Addressed in the 5th point)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

oh, thanks for spotting this bro 👍🏻 missed it out when I was editing the code haha.

@Avalionnet Avalionnet merged commit 79c7ccb into AY2021S1-CS2103T-T12-4:master Oct 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type.Documentation 📝 Updates to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants