Open nus-se-script opened 1 year ago
We disagree with the tester's point of "irrelevant details being shown to the target user" when referring to placeholder values as these indicate that the fields have not been filled in (equivalent to a field left blank). The ability to leave fields as their default values in fact reinforces relevance since there is no requirement to enter irrelevant information. Additionally, they do not prevent the relevant information from being displayed to the user once that information has been entered.
We also disagree that "realtors... will have to memorise or refer to [the] user guide for default values". For reference, the current default values include "123" as a phone number, "Placeholder Street, Singapore" as an address, and "default@email.com" as an email, etc. which are sufficiently characteristic of being placeholder values and the user should be able to intuitively identify this upon close inspection.
Hence this implementation does not contradict the user story that states realtors ought to be able to keep track of their clients' relevant details.
--
Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.
Developer guide proposes must-have user story of realtors being able to keep track of relevant details via adding users and addresses to their app. (#2 in list)
However, implementation results in irrelevant details being shown to the target user (default field values, which are meaningless and thus irrelevant).
This will inconvenience realtors who will have to memorise or refer to user guide for default values in order to realise which fields of buyers/sellers are relevant and which are not. Hence must-have user story in developer guide is not fulfilled by implementation of existing feature.
[original: nus-cs2103-AY2324S1/pe-interim#1735] [original labels: severity.Low type.DocumentationBug]