Masaki Komagata

Results 232 comments of Masaki Komagata

@reckyy コメントに返信させていただきました~

@reckyy conflictの修正をお願い致します~。

@reckyy conflictの修正をお願い致します~。

@omochiumaiumai 不要なマージPRなどが入ってしまっているようなので、mainの最新でrebaseしてみてください~

@wata00913 > 案1 > 適切なオプションで画像加工がされたか判断することが出来ない。 これってどういう意味でしょうか? > Ogp この画像リサイズはOgp画像に限らない画像変換なのでOgpのネームスペースにあるのはおかしいかもと思いました。 > ImageProcessor Processorだと意味が広すぎるのでもっと具体的な名前(例えばResizerとか?)でいいかもと思いました。

@wata00913 こちらいかがでしょうか?

@wata00913 > 良い名前が思いつかなかったため、`Ogp`のネームスペースを削除し、`ImageResizer`が適切だと思いましたが、いかがでしょうか? いいと思います〜!

@wata00913 返信を忘れており、大変申し訳ありません。 申し訳ないですが上記conflictの修正をお願いできればありがたいです。

@kyokucho1989 > 画像変換のライブラリが変更となるので、このIssueに影響があるのではとコメントしました。 はい、影響ありますね。 申し訳ないです > @wata00913 > 私も下記Issueを担当しますが、本Issueで画像切り取りのコードが共通化されそうなので、これを待って実装しようかと思っています。 それで良いと思います。 @kyokucho1989 @wata00913 もしかしたら下記で話している内容(複数種類の変換を一度に書ける)で簡単になる部分があるかもなので結果をお待ちいただければと思います。 https://github.com/fjordllc/bootcamp/pull/7397

@wata00913 こちらで @yocchan-git さんがおっしゃってるやり方でいけますでしょうか。 https://github.com/fjordllc/bootcamp/pull/7397#issuecomment-1980163313