search

securespider / pe

0 stars 0 forks source link

Delivery and index not properly explained #23

Open securespider opened 1 year ago

securespider commented 1 year ago

Are they referring to delivery riders, delivery parcel, delivery person

Very confusing as sometimes INDEX is just used and sometimes delivery index is specified

image.png

nus-se-script commented 1 year ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Displayed delivery list is unclear

UG specifies that the index of add_pc should be index shown for "displayed delivery list" but no indication on the app what that list is

image.png

image.png

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

Their Response to the 'Original' Bug

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

Might not have explicitly stated it, but should be self explanatory as there are clear numbers for index shown in list.

image.png

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 explanation]

## :question: Issue response Team chose [`response.NotInScope`] - [x] I disagree **Reason for disagreement:** ![image.png](https://raw.githubusercontent.com/securespider/pe/main/files/a2f269d7-1093-4c7f-a03a-27696e731e95.png)
This issue is in scope because it affects the usability of the app by everyday users.
## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** Consider changing the severity to low instead of verylow because it is definitely not cosmetic but may affect some users causing inconvenience The use of other package numbers in each individual person could lead users to assume that it is referring to the package index (delivery index) in the user guide. ![image.png](https://raw.githubusercontent.com/securespider/pe/main/files/1b30f3f4-7e4a-49dd-b138-790f2623f431.png)
This is clearly an issue that causes inconvenience to some if not most users because users have to triage and manually test (like i did) to figure out how to use the application properly, considering that this `index` was used in multiple features (mark parcel, mark and add).
This is unexpected because as developers, it may be self-explanatory that the index would be the person index, but it is not to everyday users.