superkaiba / pe

0 stars 0 forks source link

Beginning of "Add new Secret" section of user guide wrongfully states that the only types of data that can be stored in the application are passwords #8

Open superkaiba opened 1 year ago

superkaiba commented 1 year ago

The beginning of "Add new Secret" section of user guide wrongfully states that the only types of data that can be stored in the application are passwords

All occurrences of the word "password" in the following screenshot should be changed to "secret" so as to be more accurate with respect to the functionality of the application:

image.png

nus-pe-bot commented 1 year ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

User guide introduction says that this application is only to store passwords but it actually can store many other types of data

The user guide introduction says that this application is only to store passwords but it actually can store many other types of data.

This introduction should therefore be updated to mention that this application is for storing many different types of data.


[original: nus-cs2113-AY2223S2/pe-interim#2528] [original labels: severity.Low type.DocumentationBug]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

This is the introduction in question: Screenshot 2023-04-17 at 4.08.58 PM.png

For the purpose of our project we generally use the term "Secret(s)" or "Password(s)" to describe a general form of sensitive information that users would want to store, keep, protect and retrieve. I think the wording "Password" was not strictly defined merely as what we call a "basic password", neither is it singular. Either way, the wording used was never that it could store only Basic Passwords, but it could be store a plethora of credentials/ secrets/ passwords which also includes passwords.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your reason]


## :question: Issue response Team chose [`response.Rejected`] - [ ] I disagree **Reason for disagreement:** [replace this with your reason]