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

Able to edit nothing in Journal entry and does not display an error / explicitly say that this is possible in the UG (similar bug for editing contacts) #6029

Open
nus-pe-bot opened this issue Nov 13, 2020 · 1 comment

Comments

@nus-pe-bot
Copy link

nus-pe-bot commented Nov 13, 2020

Steps to Reproduce:

  1. Open App
  2. Enter "editj 9 n/d2" where the 9th item is already named d2 (editc behaves similarly as well)

Expected Behavior: The app displays an error message that there needs to be a difference before and after editing OR the UG explicitly explains that this behavior is condoned

Actual Behavior: The app displays that it has "Edited Journal Entry" which is not an accurate message as no fields have been edited.

  • Can be considered as a FunctionalityBug or DocumentationBug

6.PNG


[original: nus-cs2103-AY2021S1/pe-interim#6025]
@jazerler
Copy link
Contributor

Team's Response

Technically speaking, the entry has been edited; the title field has been replaced with another title that happens to be of the same string. The constraint in the UG refers to having no fields specified, and does not extend to this case. However, this has not been properly clarified, and is accepted as very low severity.

Duplicate status (if any):

--

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants