Code Review #333
完了
permissionの名前を他と被らないような名前にした方が良い
Haru Iida さんが約14年前に追加.
ほぼ14年前に更新.
説明
permissionの名前は"index"だと他のプラグインと重なるかもしれないのでもう少し長くした方が良いと思います。
さらに言うと参照権と回答権を分けると良いかもしれません。ゲストは参照だけで回答させたくない場合があるかも。
view_joel_testとanswer_joel_testみたいな感じでしょうか。
関連するチケット
2 (0件未完了 — 2件完了)
Haru Iida は書きました:
さらに言うと参照権と回答権を分けると良いかもしれません。ゲストは参照だけで回答させたくない場合があるかも。
view_joel_testとanswer_joel_testみたいな感じでしょうか。
活動に表示することなどを考えると、参照、回答権は分けておくのがよいです。
その場合 permission の名前は、 view_[module名] になります。
なので、飯田さんの view_joel_test に一票。
- ステータス を 新規（New） から 担当（Assigned） に変更
- 対象バージョン を 0.0.4 にセット
- 予定工数 を 2:00時間 にセット
- ステータス を 担当（Assigned） から 解決（Resolved） に変更
- 進捗率 を 0 から 100 に変更
- ステータス を 解決（Resolved） から 担当（Assigned） に変更
- 進捗率 を 50 から 80 に変更
プラグインチュートリアルを参考に対処致しましたが、以下のバグを組み込んでしまいました。
回避策を考えますが、しばらくかかる見込みです。
- ログインせずに回答ボタンを押下すると、ログイン画面に遷移するが、そこでログインすると、ジョエルテストの回答が全てNo扱いになってしまう。
permissionが無い場合には回答ボタンを表示しないようにすればよいのではないでしょうか。
- ステータス を 担当（Assigned） から 解決（Resolved） に変更
- 進捗率 を 80 から 100 に変更
Haru Iida は書きました:
permissionが無い場合には回答ボタンを表示しないようにすればよいのではないでしょうか。
permissionが無い場合には回答ボタンを disable にする方向で対応致しました。
- ステータス を 解決（Resolved） から 終了（Closed） に変更
- 予定工数 を 2:00時間 から 2:45時間 に変更
他の形式にエクスポート: Atom
PDF
