Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show acquired score with the message of "Correct!" #129

Open
exKAZUu opened this issue Apr 30, 2013 · 12 comments
Open

Show acquired score with the message of "Correct!" #129

exKAZUu opened this issue Apr 30, 2013 · 12 comments
Assignees

Comments

@exKAZUu
Copy link
Member

exKAZUu commented Apr 30, 2013

When a submission is correct, our app should show the message of "Correct!" with acquired score.

@ghost ghost assigned daicche Apr 30, 2013
@ss0201
Copy link
Member

ss0201 commented Apr 30, 2013

You might already know this, but I'll tell you that this issue is related to #81 and #99 (they are not merged yet), and #55.

To make things clear, I'll explain the history of #55. #55 is an issue opened by me, which at first meant "showing like 150/200 at the scores field in the problem list, which means that you'll get 150 points if you solve this problem right now and the maximum score of this problem is 200." But probably it was misunderstood to be "showing the score you've got when you solve a problem, together with Correct! or Wrong.." However, there was #9, which probably means the same thing as #55 (not quite sure, because I don't completely understand the meaning of the title.) So I closed #55.

Therefore, I think it is reasonable to relate this issue with #81 and #99, which needs further development.

@exKAZUu
Copy link
Member Author

exKAZUu commented Apr 30, 2013

I completed to develop show scores like 150/200 on the scoreboard and submission history.
So I'd like to close #9 when my pull request is accepted.

Hmm, how do you think about "showing the score you've got when you solve a problem, together with Correct! or Wrong.." ?
This issue indicates the misunderstood content (I want).

@exKAZUu
Copy link
Member Author

exKAZUu commented Apr 30, 2013

I think #129 is related to #68, #81 and #99.

@exKAZUu
Copy link
Member Author

exKAZUu commented Apr 30, 2013

BTW, miscommunication is a real problem we are facing. Interesting!

@ss0201
Copy link
Member

ss0201 commented Apr 30, 2013

Hmm, how do you think about "showing the score you've got when you solve a problem, together with Correct! or Wrong.." ?
This issue indicates the misunderstood content (I want).

Sorry, I don't understand what you're trying to say. Could you explain it in Japanese?

@ss0201
Copy link
Member

ss0201 commented Apr 30, 2013

BTW, miscommunication is a real problem we are facing. Interesting!

true :P

@exKAZUu
Copy link
Member Author

exKAZUu commented Apr 30, 2013

I think we should implement a new feature to show Correct! or Wrong with acquired score (150/200). Do you agree?

@ss0201
Copy link
Member

ss0201 commented Apr 30, 2013

Yeah. Along with that, we also need the following features: show the maximum and the current score in the problem list. Current means that what points you'll get for the problem if you solve it right now.

@ss0201
Copy link
Member

ss0201 commented Apr 30, 2013

Ah huh, now I've got what you mean. I'll just make another issue which is about my suggestion.

@exKAZUu
Copy link
Member Author

exKAZUu commented Apr 30, 2013

Thanks! I wanted to confirm we can both features about showing them on scoreboard and history, and showing them on pop up of Correct! and Wrong.
Oaky, this issue teats the latter feature.

@ss0201
Copy link
Member

ss0201 commented Apr 30, 2013

For other people, I'll put a link to the issue I opened: #131

@exKAZUu
Copy link
Member Author

exKAZUu commented May 5, 2013

Hey daich, can you finish this issue until the 7th?

@ghost ghost assigned kojisuke May 7, 2013
@ss0201 ss0201 removed this from the 1.8 (for beta test) milestone May 8, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants