Actions
Code Review #333
closed
permissionの名前を他と被らないような名前にした方が良い
Description
permissionの名前は"index"だと他のプラグインと重なるかもしれないのでもう少し長くした方が良いと思います。
さらに言うと参照権と回答権を分けると良いかもしれません。ゲストは参照だけで回答させたくない場合があるかも。
view_joel_testとanswer_joel_testみたいな感じでしょうか。
Updated by Toshiyuki Ando over 14 years ago
Haru Iida は書きました:
さらに言うと参照権と回答権を分けると良いかもしれません。ゲストは参照だけで回答させたくない場合があるかも。
view_joel_testとanswer_joel_testみたいな感じでしょうか。
活動に表示することなどを考えると、参照、回答権は分けておくのがよいです。
その場合 permission の名前は、 view_[module名] になります。
なので、飯田さんの view_joel_test に一票。
Updated by Takashi Takebayashi over 14 years ago
- Status changed from 新規（New） to 担当（Assigned）
- Target version set to 0.0.4
- Estimated time set to 2:00 h
Updated by Takashi Takebayashi over 14 years ago
- Status changed from 担当（Assigned） to 解決（Resolved）
- % Done changed from 0 to 100
更新履歴 r23 で適用されました。
Updated by Takashi Takebayashi over 14 years ago
- Status changed from 解決（Resolved） to 担当（Assigned）
- % Done changed from 50 to 80
プラグインチュートリアルを参考に対処致しましたが、以下のバグを組み込んでしまいました。
回避策を考えますが、しばらくかかる見込みです。
- ログインせずに回答ボタンを押下すると、ログイン画面に遷移するが、そこでログインすると、ジョエルテストの回答が全てNo扱いになってしまう。
Updated by Haru Iida over 14 years ago
permissionが無い場合には回答ボタンを表示しないようにすればよいのではないでしょうか。
Updated by Takashi Takebayashi over 14 years ago
- Status changed from 担当（Assigned） to 解決（Resolved）
- % Done changed from 80 to 100
Haru Iida は書きました:
permissionが無い場合には回答ボタンを表示しないようにすればよいのではないでしょうか。
permissionが無い場合には回答ボタンを disable にする方向で対応致しました。
Updated by Takashi Takebayashi over 14 years ago
- Status changed from 解決（Resolved） to 終了（Closed）
Updated by Takashi Takebayashi over 14 years ago
- Estimated time changed from 2:00 h to 2:45 h
Actions