search

hpcman / pe

0 stars 0 forks source link

Add module and role command error does not specify the structure of module code #2

Open hpcman opened 1 week ago

hpcman commented 1 week ago

3.png Input: edit 12 r/CS The error message should specify the expected input of module code since other universities may use different code.

soc-se-bot commented 1 week ago

Team's Response

Thanks for pointing it out, but this is intended because:

  1. Message cannot be too complicated.

Also, your suggestion that

The error message should specify the expected input of module code since other universities may use different code.

is not really applicable because we have set our target users to be NUS students specifically, as mentioned in the UG (and DG).

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Thank you for your response. As mentioned in the UG, the application also supports DYOC which by current batch also support edX modules that can be based on other university codes, e.g. CS50. The current DYOC also support AISG which don't have specified module code. There might be other DYOC module code that does not follow the UG module code format.