プロジェクト

全般

プロフィール

Code Review #333

完了

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

Haru Iida さんがほぼ14年前に追加. ほぼ14年前に更新.

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

100%

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

説明

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

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

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


関連するチケット 2 (0件未完了2件完了)

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

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

操作

Toshiyuki Ando さんがほぼ14年前に更新

Haru Iida は書きました:

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

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

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

Takashi Takebayashi さんがほぼ14年前に更新

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

Takashi Takebayashi さんがほぼ14年前に更新

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

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

Takashi Takebayashi さんがほぼ14年前に更新

  • 進捗率100 から 50 に変更

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

Takashi Takebayashi さんがほぼ14年前に更新

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

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

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

Haru Iida さんがほぼ14年前に更新

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

Takashi Takebayashi さんがほぼ14年前に更新

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

Haru Iida は書きました:

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

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

Takashi Takebayashi さんがほぼ14年前に更新

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

Takashi Takebayashi さんがほぼ14年前に更新

  • 予定工数2:00時間 から 2:45時間 に変更

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