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

数値の答えにマイナスの数を入力しても、UIにエラーが出ない #145

Open
mekpsy opened this issue Sep 25, 2018 · 6 comments
Assignees

Comments

@mekpsy
Copy link
Contributor

mekpsy commented Sep 25, 2018

数値の答えにマイナスの数を入力しても、UIにエラーが出ない。

web3-10 / master 両方

@mekpsy
Copy link
Contributor Author

mekpsy commented Sep 25, 2018

質問の投稿では、Number(Positive)と書いてありますが、答えをゼロにしてもエラーにはならないようです。MetaMaskが立ち上がります。

@mekpsy
Copy link
Contributor Author

mekpsy commented Oct 11, 2018

数字で入力するところが、数値の質問の答えのほかにもいくつかあります。
main.jsでは同じところで処理をしているので、整理したいから質問させてください。

それぞれの仕様は、次の通りでいいでしょうか?

答えの数字
仕様:Positiveであること
現状:正しく動いている。
疑問:ゼロの答えは必要ない?

質問を作るときのReward
仕様:ゼロ以上であること
現状:Formの中でゼロを入力するとエラーメッセージが表示される。Submitは行われます。

Rewardの追加
仕様:Positiveであること
現状:正しく動いている

Bondの設定
仕様:最初の答えのときはゼロ以上であること。2回目以降は、前のBondの2倍以上であること。
現状:正しく動いている

@mekpsy
Copy link
Contributor Author

mekpsy commented Oct 11, 2018

Spaceを数値のところに入力するとUIでエラーが表示されない不具合があります。
これは直しておきました。

@mekpsy
Copy link
Contributor Author

mekpsy commented Oct 11, 2018

ああ、UIでエラーは表示されるようにしたけど、BondだけはSpaceを入力でもSubmitされてしまう。
これも直します。

このIssueのコードはコミットしてないです。Edgarさんからコメントの答えをもらってからやります。

@edmundedgar
Copy link
Contributor

edmundedgar commented Oct 12, 2018

答えの数字
仕様:Positiveであること
現状:正しく動いている。
疑問:ゼロの答えは必要ない?

ゼロの答えは入力可能であるべき、しかもディフォルト

質問を作るときのReward
仕様:ゼロ以上であること
現状:Formの中でゼロを入力するとエラーメッセージが表示される。Submitは行われます。

ゼロのRewardは入力可能であるべき、しかもディフォルト

Rewardの追加
仕様:Positiveであること
現状:正しく動いている

ok

Bondの設定
仕様:最初の答えのときはゼロ以上であること。2回目以降は、前のBondの2倍以上であること。
現状:正しく動いている

ok

@mekpsy
Copy link
Contributor Author

mekpsy commented Oct 19, 2018

対応しました。

4faa66b

数値で答える問題を作るとき、question typeのところに(Positive)とあるので、(>= 0) とかになおした方がいいかも。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants