eliztan / pe

0 stars 0 forks source link

Favourites list does not show inputted Legend keywords #6

Open eliztan opened 2 months ago

eliztan commented 2 months ago

image.png

When inputting a favourite, whether it be activity or food, there are required location/price/cuisine keywords to be inputted as well. However, they are not present in the favourites list, potentially leaving the user confused when they return to their favourites list as the keywords are not there.

Steps to replicate:

  1. Run the activity or food command to add an item to favourites.
  2. Run the favourites command.
nus-pe-script commented 2 months ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Favourites does not specify Location and Price

The favourites list does not print the location nor price of the activity. Unless it is used in personalisation such as in smart (if so, it is not mentioned in the userguide), it makes the whole point of entering in a location and price pointless:

____________________________________________________________
favourites
Drumroll, please! Presenting the stars of your romantic sky:
1. run
____________________________________________________________

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

Their Response to the 'Original' Bug

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

Thank you for raising up this issue! To address some of your concerns:

1. Issue: "favourites list does not print the location nor price of the activity" Given that users are responsible for the description of the activity when inputting 'favourites', we believe that the responsibility is on the user to ensure that there is enough distinction to understand what their favourited activity is.

Citing the example you gave, if the user decided to favourite their activity as 'run', it implies that the user simply enjoys running, and thus data such as the location of the activity is redundant due to the very broad nature of the activity 'run'. We believe that it is a fair assumption to assume that for a meaningful 'favourite' entry, users would be more specific and personal, such as entering 'run at East Coast Park'. In this case, the location and price of the activity can be inferred from the description itself.

While we acknowledge that moving forward, displaying the location and price point would provide some clarity to the users, we believe that this issue is of lower priority than the basic functionality that the team was trying to achieve in v2.1

2. Issue: "it makes the whole point of entering in a location and price pointless"
The purpose of asking users to include a location and price when inputting 'favourites' as well is that not only is the activity updated in the 'Favourites' database, but it is updated in the 'ActivityList' database as well. While the 'location' and 'price' currently serve little function within the 'favourites' command, these inputs are necessary to be included in the 'ActivityList' database, as they will be used to filter out the relevant activities during generation of idea or itinerary

Screenshot 2024-04-22 120423.png

Screenshot 2024-04-22 120449.png

However, we believe that your feedback is valid to be used in improvements for future iterations and hence, will mark it as NotInScope.

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`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue severity Team chose [`severity.Medium`] Originally [`severity.Low`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]