AI-comp / wpcs

2 stars 2 forks source link

Failed count is not required? #105

Closed exKAZUu closed 11 years ago

exKAZUu commented 11 years ago

Currently, users can't know how many times they failed. If this feature is not require, we should remove "solved_at" field on Submit class.

ss0201 commented 11 years ago

Oh, actually you can. "Solved time" field in the "Problem list" shows like "Not yet (2 failures)." And I think it's better to show it.

ss0201 commented 11 years ago

Forget the last statement. It might be better not to show it, so that they won't be afraid of making failed submissions.

exKAZUu commented 11 years ago

Okay! On Apr 29, 2013 7:38 PM, "ss0201" notifications@github.com wrote:

Forget the last statement. It might be better not to show it, so that they won't be afraid of making failed submissions.

— Reply to this email directly or view it on GitHubhttps://github.com/TakenokoChocoHolic/wpcs/issues/105#issuecomment-17159737 .

exKAZUu commented 11 years ago

BTW, should the submission history show failed submissions?

exKAZUu commented 11 years ago

Currently, our app does not record failed submissions. Is it OK?

ss0201 commented 11 years ago

Oh.. this is not good. If I were a contest attendant, I would definitely want to see failures.

exKAZUu commented 11 years ago

Okay, I will record all submissions.

Summarizing the above, we don't show the number of failed submissions but we show all the submissions including failed one.

ss0201 commented 11 years ago

Yes.