r-labs » Work Time

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

時期は問題ありません。

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

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

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

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

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

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

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

実際に連携させてみました。(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を用意したほうが良ければ、ご連絡ください。

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

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

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

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

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

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

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

お久しぶりです。

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