search

Coding-Trip / CodingTripAndroid

CodingTripのAndroidクライアント
https://xd.adobe.com/view/54acd9e3-4978-482e-a0ac-137f89a4e458/
4 stars 0 forks source link

Dagger追加 #16

Closed halu5071 closed 6 years ago

halu5071 commented 6 years ago

Kodeinがはじめ入ってたけど、勉強不足でまだ使いこなせてないです。

このせいで進捗遅くなるよりDagger実装のほうがいいかなと思ったので、DaggerでのPR出します。

Kodeinのほうがいいならリジェクトしてください。

omuomugin commented 6 years ago

まあ、わからなければリファクタリングはあとでするー

omuomugin commented 6 years ago

コミットだけ rebaseしてまとめてー

halu5071 commented 6 years ago

fix commit をまとめました @omuomugin お願いしまっす これが Approveされたら #9 を一旦closeします

omuomugin commented 6 years ago

間違えた

halu5071 commented 6 years ago

これわざわざする意味あります？？ よくわかってないです

omuomugin commented 6 years ago

ここのInterfaceが呪文的なのが気になる。

halu5071 commented 6 years ago

確かに嫌ですね

omuomugin commented 6 years ago

PR送る

omuomugin commented 6 years ago

いろいろコード見てたけど、DroidKaigiに乗っ取るのが良さそうだな。

omuomugin commented 6 years ago

マージしたからこれもマージで