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

デバッグモードでのエラーチェック #97

Closed
snext1220 opened this issue Feb 4, 2019 · 8 comments
Closed

デバッグモードでのエラーチェック #97

snext1220 opened this issue Feb 4, 2019 · 8 comments
Labels
enhancement 新機能提案* wontfix 保留(バグ報告に問題がなかった or 機能提案見送りなど)

Comments

@snext1220
Copy link
Owner

snext1220 commented Feb 4, 2019

  • デバッグモード時のエラーチェックを強化(主に、init要素など新機能)
  • New Playgroundのデバッグモード対応

ご執筆時に間違えやすい、またはチェックが難しいなどのポイントがあれば、自動チェックの項目を増やしていければと。ご要望あればお願いします(現在も一部はチェックしていますが、すべては困難なので、ご要望のあるものから対応していきます)。

@snext1220 snext1220 added the enhancement 新機能提案* label Feb 4, 2019
@snext1220 snext1220 added this to the May 2019 Update milestone Feb 24, 2019
@snext1220 snext1220 removed this from the May 2019 Update milestone Mar 3, 2019
@snext1220 snext1220 added the Expired soon 開始から時間経過しているので、近日Close、またはConfirmed昇格 label Mar 10, 2019
@snext1220 snext1220 added this to the May 2019 Update milestone Mar 10, 2019
@snext1220 snext1220 added the wontfix 保留(バグ報告に問題がなかった or 機能提案見送りなど) label Mar 17, 2019
@toki-sor1
Copy link
Collaborator

私が把握していないだけで、既に実装されていましたらすみません;
シーンの「アイテム」などで登録していないアイテムを設定すると、テストプレイ時に警告が出るのですが、リンクの「条件式」に登録していないアイテムなどを設定した時は警告が出ませんでした。

条件式の方でも、登録情報と食い違いがあった時に警告が出ると良いかな と思いました。
#ただ、そうするとドラモードのように違うシナリオの実績を指定した時に警告が出てしまったりするのでしょうか?(その場合は現状ままでお願いします~)

@snext1220
Copy link
Owner Author

ご要望ありがとうございます!

はい、現在はご認識の通り、条件式に含まれるアイテムの不整合はノーチェックです。
属性と本文(条件式)は全く別物のしくみで、しかも本文の解析は手作業なので現時点では対象外としていました^^;

まだどういう形で実装していくかは未確認なのですが、まずは調査してみますので、暫しお待ちくださいませ。

#ということで、本件はwontfixを外し、June以降の対応としていきますね。

@snext1220 snext1220 removed the wontfix 保留(バグ報告に問題がなかった or 機能提案見送りなど) label Apr 7, 2019
@snext1220 snext1220 added Confirmed 議論終了&実装待ち(実装時に仕様修正の可能性あり) and removed Expired soon 開始から時間経過しているので、近日Close、またはConfirmed昇格 labels Apr 14, 2019
@snext1220 snext1220 removed this from the June 2019 Update milestone Apr 14, 2019
@snext1220 snext1220 added wontfix 保留(バグ報告に問題がなかった or 機能提案見送りなど) and removed Confirmed 議論終了&実装待ち(実装時に仕様修正の可能性あり) labels Feb 29, 2020
@snext1220
Copy link
Owner Author

本件ですが、PgFlowの入力支援機能によって条件式の入力誤りを防ぎやすくなった点から、必要性が低下したように思えます。一旦、[Confirmed]にしたIssueではありますが、問題なければ[wontfix]にできればと思いますが、如何でしょうか。

# シナリオ作成時に間違いを引き起こしやすいなどの問題については、PgFlowにて対応していければと思いますので、お気軽にリクエスト願います(勿論、「いやいややはりデバッグモードで~」というご意見もありうると思っています^^;)。

@snext1220 snext1220 added this to the April 2020 Update milestone Feb 29, 2020
@toki-sor1
Copy link
Collaborator

大変遅くなってしまい申し訳ありません;
仕様についてご説明いただき、また、実装についてもご検討いただき、ありがとうございました!

属性と本文のしくみは別物であることや、入力支援機能の実装で誤入力を防ぎやすくなったことから、本件は[wontfix]にしていただいて問題ありません!

# シナリオ作成時に間違いを引き起こしやすいなどの問題については、~

ありがとうございます!
これは間違いを起こし易そうかなと感じるような箇所がありましたら、その時はまたご報告やリクエストさせていただきますね~。

@snext1220
Copy link
Owner Author

ご了解戴き、ありがとうございます~
PgFlowにリソースを集中させるために、近頃結構wontfixしてますが、勿論他のツール、システム本体も引き続き強化していきますので、リクエストなんなりとです!

@cocotori
Copy link
Collaborator

cocotori commented Mar 7, 2020

すみません。そもそものところがわかっておらず申し訳ないです・・・
デバッグモードというのがよくわからないのですが、
どういった機能なのですか?

playgroundの画面から、なにか操作をするとデバッグモードに
移行するのでしょうか。

@snext1220
Copy link
Owner Author

Playgroundでのデバッグ支援機能になります。
詳しくは以下を参照戴けますと幸いですm(_ _)m

https://sorcerian.hateblo.jp/entry/20180125
[5]デバッグモードを利用する

@cocotori
Copy link
Collaborator

cocotori commented Mar 7, 2020

動作確認したときに上に表示されている枠のことだったのですね。
よく役立たせてもらっています。

エラーチェックについてはそうですね、
入力支援機能がありますので
wontfixで問題ありません!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement 新機能提案* wontfix 保留(バグ報告に問題がなかった or 機能提案見送りなど)
Projects
None yet
Development

No branches or pull requests

3 participants