プロジェクト

全般

プロフィール

Feature #1362

プラグイン拡張のためのHTML属性の追加

Akira Saitoほぼ3年前に追加. 2年以上前に更新.

ステータス:
終了(Closed)
優先度:
通常(Normal)
担当者:
対象バージョン:
開始日:
2015/01/07
期日:
進捗率:

100%

予定工数:

説明

システム開発でWork Timeを利用していますが、
チケットの名称が被ってしまい、識別のために詳細を見るといったことが、時々あります。

私はQuick Viewというプラグインを公開していますが、
このプラグインではチケットの詳細を(jQueryの)ダイアログ表示することができるので、
連携できると少し便利になります。

具体的には、cssのclassを各要素に追加したり、HTMLのカスタムデータ属性を追加したり(例えばdata-issueでチケット番号を識別可能にする)
して頂けると、連携しやすいと考えています。

よろしければパッチを見ていただけませんでしょうか?

_user_day_table.html.erb.patch (662 Bytes) _user_day_table.html.erb.patch Akira Saito, 2015/01/07 22:49

関係しているリビジョン

リビジョン 378:d5921b94a879 (差分)
Tomohisa Kusukawaほぼ3年前に追加

fixes #1362 プラグイン拡張のためのHTML属性の追加

リビジョン 385:b37efd09dd08 (差分)
Tomohisa Kusukawa2年以上前に追加

fixes #1362 プラグイン拡張のためのHTML属性の追加

リビジョン 386:7ffa6369c961 (差分)
akira saito が2年以上前に追加

fixes #1362 プラグイン拡張のためのHTML属性の追加

履歴

#1 Tomohisa Kusukawaほぼ3年前に更新

ご連絡ありがとうございます。
軽微な変更で副作用もないと思いますので
採用させて頂きたいと思います。
ただ最近時間がなかなか取れず、少しお待ち頂くと思いますが、ご容赦頂けますでしょうか。

#2 Akira Saitoほぼ3年前に更新

時期は問題ありません。

よろしくお願い致します!

#3 Tomohisa Kusukawaほぼ3年前に更新

  • ステータス新規(New) から 解決(Resolved) に変更
  • 進捗率0 から 100 に変更

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

#4 Tomohisa Kusukawaほぼ3年前に更新

  • 担当者Akira Saito にセット

対応が遅く大変恐縮です。

d5921b94a879 でパッチの内容を(手動で)適用しましたので、
確認して頂ければと思います。

ちょっと思ったのは、
① 'data-issue' のハイフンはアンダーバーにしてシンボル :data_issue にした方が良くないか
② <%= issue_id %> はダブルクォートで囲んだ方が良いのではないか
なのですが、如何でしょうか。

#5 Akira Saitoほぼ3年前に更新

ご対応ありがとうございます。返信が遅れてすみません。
ローカルで確認したいので、少し待ってください。

ご指摘については以下のように考えています。

① 'data-issue' のハイフンはアンダーバーにしてシンボル :data_issue にした方が良くないか

 HTML5で、data-*属性が独自属性として正式に利用できるようになり、
 DOMやjQueryもAPIを備えていますが、
 WorkTime Pluginが外のPluginと連携するのであれば、
 より標準的なI/Fを利用するほうが良いと思い、この形で提案させていただきました。
 こちらとしては、(URLをパースする以外の方法で)チケットのIDが取れさえすれば不都合はありません。

② <%= issue_id %> はダブルクォートで囲んだ方が良いのではないか

 こちらについてはその通りだと思います。

#6 Akira Saitoほぼ3年前に更新

実際に連携させてみました。(git://git.sourceforge.jp/gitroot/quickedit/quick_view.git)

試したところ、Daily Reportでチケット工数ありの場合に、
wt_done_ratioクラス属性が無く、連携が動作しませんでした。
元々のパッチになかったのが問題だったのですが、追加頂くことは出来ますか?

それと、'data-issue'と、:'data-issue'という記述が混在しているので、
統一すべきかなと思いました。

また、add ticketのsubmit時に生成されるチケット行にも、
クラス属性とdataxxx属性(data-にするかdata_にするかはお任せします)が欲しいです。
できればmonthly reportの方にも。。。
patchを用意したほうが良ければ、ご連絡ください。

#7 Tomohisa Kusukawa2年以上前に更新

ご確認頂き、ありがとうございます。
# 対応が遅くて恐縮です。

もろもろあるので、具体的なコードで示して頂けると助かります。
できれば、パッチよりもPull Requestの方が助かるのですが、
(貢献も明確になりますし)ハードルが高い用であればパッチでも大丈夫です。

#8 Akira Saito2年以上前に更新

それではPull Requestで試してみます。
Pull Requestは初めてなのと、別の作業もしているので、ちょっと遅れます。

#9 Tomohisa Kusukawa2年以上前に更新

時間ができましたので対応してみます。
コミット後にまたご確認をお願い出来ますでしょうか。

#10 Akira Saito2年以上前に更新

プルリクストを送りましたが、かぶってしまった上に、ミスっています。
取り消せないのでしょうか。。。

日毎工数表について、問題が解消されたことを確認しました。

2度目のプルリクエストには、月次レポートの修正要望も含めています。
採用いただければ嬉しいです。

#11 Tomohisa Kusukawa2年以上前に更新

  • 対象バージョン0.3.0 にセット

お願いしてたのに、かぶってしまい申し訳ありませんでした。

最後のプルリクエストを取り入れさせて頂きました。

#12 Tomohisa Kusukawa2年以上前に更新

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

Version 0.3.0 に含めさせて頂きますね。

#13 Akira Saito2年以上前に更新

お久しぶりです。

こちらもQuick View 0.1.0で、連携機能をリリースさせていただきました。
ご協力ありがとうございました。

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