プロジェクト

全般

プロフィール

Code Review #333

permissionの名前を他と被らないような名前にした方が良い

Haru Iidaほぼ8年前に追加. ほぼ8年前に更新.

ステータス:
終了(Closed)
優先度:
通常(Normal)
対象バージョン:
開始日:
2010/04/15
期日:
進捗率:

100%

予定工数:
2.75時間
作業時間:
コードレビュー: /init.rb@21:line 27

説明

permissionの名前は"index"だと他のプラグインと重なるかもしれないのでもう少し長くした方が良いと思います。

さらに言うと参照権と回答権を分けると良いかもしれません。ゲストは参照だけで回答させたくない場合があるかも。

view_joel_testとanswer_joel_testみたいな感じでしょうか。


関連するチケット

関連している Joel Test - Defect #331: プロジェクトの設定内のモジュールタブにJoel testと表示されている。終了(Closed)2010-04-15

次のチケットが重複 Joel Test - Feature #332: ジョエルテストに答えられるのはログインユーザに限定する。終了(Closed)2010-04-15

関係しているリビジョン

リビジョン 23:18870c109e8e (差分)
Takashi Takebayashiほぼ8年前に追加

fixes #333
permission index -> view_joel_test へ変更

リビジョン 24:685e010af094 (差分)
Takashi Takebayashiほぼ8年前に追加

refs #333
permission answer_joel_test 追加
permission に合わせて、action を変更
不要な""部分を''に変更

リビジョン 25:4ead65aee1a0 (差分)
Takashi Takebayashiほぼ8年前に追加

refs #333
permission のラベル追加
permission チェックを追加

リビジョン 26:60cc2c90dac9 (差分)
Takashi Takebayashiほぼ8年前に追加

fixes #333
permission answer_joel_test が無い場合は、回答ボタンを disable に変更

履歴

#1 Toshiyuki Andoほぼ8年前に更新

Haru Iida は書きました:

さらに言うと参照権と回答権を分けると良いかもしれません。ゲストは参照だけで回答させたくない場合があるかも。

view_joel_testとanswer_joel_testみたいな感じでしょうか。

活動に表示することなどを考えると、参照、回答権は分けておくのがよいです。
その場合 permission の名前は、 view_[module名] になります。
なので、飯田さんの view_joel_test に一票。

#2 Takashi Takebayashiほぼ8年前に更新

  • ステータス新規(New) から 担当(Assigned) に変更
  • 対象バージョン0.0.4 にセット
  • 予定工数2.00時間 にセット

#3 Takashi Takebayashiほぼ8年前に更新

  • ステータス担当(Assigned) から 解決(Resolved) に変更
  • 進捗率0 から 100 に変更

更新履歴 r23 で適用されました。

#4 Takashi Takebayashiほぼ8年前に更新

  • 進捗率100 から 50 に変更

コミットキーワード不正の為、進捗変更

#5 Takashi Takebayashiほぼ8年前に更新

  • ステータス解決(Resolved) から 担当(Assigned) に変更
  • 進捗率50 から 80 に変更

プラグインチュートリアルを参考に対処致しましたが、以下のバグを組み込んでしまいました。
回避策を考えますが、しばらくかかる見込みです。

  • ログインせずに回答ボタンを押下すると、ログイン画面に遷移するが、そこでログインすると、ジョエルテストの回答が全てNo扱いになってしまう。

#6 Haru Iidaほぼ8年前に更新

permissionが無い場合には回答ボタンを表示しないようにすればよいのではないでしょうか。

#7 Takashi Takebayashiほぼ8年前に更新

  • ステータス担当(Assigned) から 解決(Resolved) に変更
  • 進捗率80 から 100 に変更

Haru Iida は書きました:

permissionが無い場合には回答ボタンを表示しないようにすればよいのではないでしょうか。

permissionが無い場合には回答ボタンを disable にする方向で対応致しました。

#8 Takashi Takebayashiほぼ8年前に更新

  • ステータス解決(Resolved) から 終了(Closed) に変更

#9 Takashi Takebayashiほぼ8年前に更新

  • 予定工数2.00時間 から 2.75時間 に変更

他の形式にエクスポート: Atom PDF