Open ziyingli opened 4 years ago
noted on the issue. we will add the relevant exception handling into our code to return a message to the user if the text is invalid. however, since this is an exceptional case, it does not disrupt the main flow of our application till the point where the "product is unusable for most users". we have reassigned this bug a severity of medium.
Team chose [severity.Medium
]
Originally [severity.High
]
Reason for disagreement: I disagree with the severity level and the team's reasoning, given the two reasons below:
The parameter provided is not invalid and this is not an exceptional case, as both the UG and the usage message displayed in the app clearly states that the e/exercise_name
can be alphanumeric (attached below). Therefore using find
with a numeric parameter should work the same way as using find
with an alphabetic parameter. This issue is not an exceptional case.
This makes the app highly unsuable as this find
function is present not only in the exercise
component, it is present in workout
and log
components of the overall app. Given that the find
function is used in 3 out of the 4 major components and all the find
commands have this problem, I believe that this makes the app highly unsuable.
UG Requirement:
App usage message:
Exercise find should return 1 exercise found but Feedback Box does not give me any response