tlyi / pe

0 stars 0 forks source link

Unnecessary warehouse shelf automatically loaded #13

Open tlyi opened 2 years ago

tlyi commented 2 years ago

Not sure if it was intended but the warehouse shelf is automatically created. Perhaps it would be a good idea to tell the user about that in the UG, as now it seems like it was unintentionally left inside the app.

soc-pe-bot commented 2 years ago

Team's Response

The “warehouse” was supposed to be renamed as “sample shelf” to help the user get started as an example.

The 'Original' Bug

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

Redundant data in storage

The program has some redundant data before the users create or add anything. This is not recommended as the users may not want to add it during execution. And users have to delete the bookshelf if they do not want it in the program.

Bug2.png


[original: nus-cs2113-AY2122S1/pe-interim#409] [original labels: type.FeatureFlaw severity.Medium]

Their Response to the 'Original' Bug

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

The warehouse shelf was supposed to be an example shelf for the new user, which could be easily removed by the user.

This is similar to GitHub project kanban. When you first create a project kanban, it automatically creates three lists with some sample entries in it.

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]

Reason for disagreement: I do not think it is appropriate to mark this as "not in scope" as it does indeed affect the user's experience using the app as of v2.1. However, I agree with the change from FunctionalityBug to FeatureFlaw since it does not affect the user's ability to use the app.


:question: Issue type

Team chose [type.FeatureFlaw] Originally [type.FunctionalityBug]

Reason for disagreement: [replace this with your explanation]