search

vimalapugazhan / pe

0 stars 0 forks source link

deleteEvent relies on delExEv was ordered wrongly #12

Open vimalapugazhan opened 2 months ago

vimalapugazhan commented 2 months ago

image.png

The deleteEvent command requires users to use the delExEv command to remove all the expenses linked to it before finally deleteing the event.

It would make sense to put delExEv section before deleteEvent

image.png

nus-pe-bot commented 2 months ago

Team's Response

The requirement and introduction of command delExEv was clearly stated in the deleteEvent description.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: All the more reason it would make sense sections 8 and 10 of the UG to be swapped. Users will be trying to use the delExEv command without properly learning how to use the delExEv coupled with the unclear explanation of delExEv, the user may make mistakes in trying to delete the event. Maybe a way to keep the current order of UG and improving the clarity of deleteEvent command would be to insert a dropdown or link to the delExEv section.