muhammad-faruq / pe

0 stars 0 forks source link

Non-intuitive choice of commands for appt -SOMETHING #5

Open muhammad-faruq opened 2 years ago

muhammad-faruq commented 2 years ago

while it makes sense to set the upcoming to appt -u, and filtering as appt -f, it makes no sense to set the list of appointments for today as appt -l. If I may, what does the "l" stand for in appt -l?

nus-pe-bot commented 2 years ago

Team's Response

appt -l stands for list today's appointment. The behavior is intended to simulate the default behavior of the appointment list on start up of the application, where the appointment list displays the current day's appointment.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Hi there,

For appt -l you stated that it stands for list. From what I understand a list command will be used to list ALL of a certain list. Hence once again this proves the point that the command is in fact no intuitive and that I should expect appt -l to in fact display ALL the appointments rather than the appointments for today. Hence the point still stands and I still believe that this should be a FeatureFlaw with Severity: Low.