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

"addVisit" command prevents addition of future dates. #376

Merged

Conversation

kaboomzxc
Copy link

@kaboomzxc kaboomzxc commented Nov 11, 2024

"addVisit" command prevents addition of future dates.
Thank you Glendon for the helpful suggestion

closes #320

Thank you Glendon for the helpful suggestion
@kaboomzxc kaboomzxc self-assigned this Nov 11, 2024
@kaboomzxc kaboomzxc added this to the v2.1 milestone Nov 11, 2024
@kaboomzxc kaboomzxc added the priority.Medium Nice to have label Nov 11, 2024
@@ -338,7 +338,8 @@ Additional examples:
• The NRIC must belong to an **<u>existing patient</u>** in the system <br>
• Date and Time format must be in : dd-MM-yyyy HH:mm <br>
• Parameters entered in the input can be of <u>any order</u> and is <u>allowed</u>, i.e. you may input "ic/", "v/", "d/", "m/" <u>in any order</u>.
Or you may also choose to stick to convention and input "ic/", "v/", "d/", "m/" in this order.
Or you may also choose to stick to convention and input "ic/", "v/", "d/", "m/" in this order. <br>
• "addVisit" does <u>not allow</u> for adding <u>future Dates&Time</u>. Consider scheduling an Appointment instead.

Choose a reason for hiding this comment

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

LGTM, nice clarification in UG

@coraleaf0602 coraleaf0602 merged commit b3ce904 into AY2425S1-CS2113-T10-2:master Nov 11, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority.Medium Nice to have
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[PE-D][Tester A] addVisit able to add records with dates in the future.
2 participants