Skip to content

Conversation

@basyura
Copy link
Contributor

@basyura basyura commented Jun 20, 2020

課題

  • 特定のプロジェクトでだけ発動させたい場合に、パスにプロジェクト識別子を指定してもチケットを開くと url からプロジェクト識別子が消えてしまって発動しなくなってしまいます。
  • パスは .* を指定し、javascript を使って画面上の要素 $(".current-project") を見て判断することもできますが、
    そうなると .* 指定ばかりになってしまうので、選択中のプロジェクト (識別子) をパターン指定できるようにしてみました。

できると僕が嬉しいこと

  • 任意のプロジェクトを選択中に任意の自作タブをプロジェクト、活動、チケットの位置に追加して表示する事ができます。自分担当や任意のクエリタブを追加したりすると便利です。

スクリーンショット 2020-06-20 21 13 36

↓ は testproject 選択中のときだけヘッダを赤色にしたい場合。

スクリーンショット 2020-06-20 21 18 42

@onozaty
Copy link
Owner

onozaty commented Jun 20, 2020

プルリクエストありがとうございます。
確認、マージして次バージョンでリリースしたいと思います。

@onozaty onozaty changed the base branch from master to develop/2.7.0 June 21, 2020 07:36
@onozaty onozaty merged commit 02a69e3 into onozaty:develop/2.7.0 Jun 21, 2020
@onozaty
Copy link
Owner

onozaty commented Jun 21, 2020

@basyura
こちらで確認したところ、挿入位置が全てのヘッダ以外だとエラーになっていましたので、 3298b70 で修正しました。

#63 の対応が終わったら2.7.0としてリリースさせていただきます。

あと今までユニットテストを避けていた(RubyやRailsの作法を覚えるのが面倒だった、、)のですが、ユニットテストを追加していただいたので、今後はテストパターンを増やしていこうと思います。
ありがとうございました。

@basyura
Copy link
Contributor Author

basyura commented Jun 22, 2020

3298b70 で修正しました。

確認不足でした。ありがとうございます _(_ _)_

あと今までユニットテストを避けていた(RubyやRailsの作法を覚えるのが面倒だった、、)

僕も同じで・・・Ruby のテストを書いたことはほぼないのですが
なるべく Rails に依存しない形で組んでみました。ガッツリ書き換えてもらって構いません。

リリースさせていただきます。

よろしくお願いします。

NiebieskiRekin pushed a commit to NiebieskiRekin/redmine-view-customize that referenced this pull request Jul 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants