search

nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

`mark` allows extra user input without warning #1697

Open nus-pe-bot opened 2 weeks ago

nus-pe-bot commented 2 weeks ago

Currently mark marks all students' attendance, but input things like mark 1 won't throw and error or warning to user. Users may input mark 1 thinking it only marks attendance of first student but marks all students. It's quite difficult for user to remember ug all the time.

Suggestion: execute command, or display a warning message please notice mark command will mark all students if user inputs an index

[original: nus-cs2103-AY2425S1/pe-interim#2335] [original labels: type.FeatureFlaw severity.Low]

AgentHagu commented 2 weeks ago

Team's Response

image.png

We mentioned earlier in the UG that extra parameters for certain commands that don't accept parameters will be ignored. About the functionality of mark 1 marking the first index's student, we've also already mentioned implementing this under Planned Enhancement 2 in our DG.

Duplicate status (if any):

--