pj-picbook / picbook

9 stars 0 forks source link

Feature/#59-themadata #68

Closed karokita closed 2 years ago

karokita commented 2 years ago

関連のタスクissue*

PR作成時のチェックリスト*

※対象外の場合は実施していなくてもチェックを入れる

対応したこと*

キャプチャ

Simulator Screen Shot - iPhone 13 - 2022-07-31 at 23 56 42

未解決の課題

その他・備考

リリース時の注意点

karokita commented 2 years ago

📝メモ プルリクはできてそうだけどcheck the source codeでひっかかってる

スクリーンショット 2022-08-01 0 09 13

tokku5552 commented 2 years ago

プルリクはできてそうだけどcheck the source codeでひっかかってる

これ対応できそうですか?わからなければ私やります!

Ta23ka98 commented 2 years ago

@tokku5552 僕は分からないのでお願いしたいです🙇

kawadon780 commented 2 years ago

@tokku5552 レビューしてみました。こんな粒度で問題ないでしょうか?あとなんかレビューの通知がSlackに飛ばない?ぽいかもですね。。。

karokita commented 2 years ago

@tokku5552 @Ta23ka98 @kawadon780 確認ありがとうございます!check the source codeの内容、下記だったようなので対応しました。

prefer_interpolation_to_compose_strings 文字や値からなる文字列を作成するときに補完($)を使用することを推奨

修正後のものをプッシュしたところ、エラーも消えたみたいです!(→消えてなかった)

かわどさんのレビューの記録がなくて、もしかしたらレビュー残ってないのかも...?という気もしました。 レビュー内容が気になるので、そちら確認して問題ないとなり次第、マージ対応したいと思います!

karokita commented 2 years ago

@tokku5552 ~~エラー消えたと思いきや、消えてませんでした🤔 コード修正した後の対応方法教えていただけるとありがたいです!~~

→こちらボトムナビも同様の対応したところいけました。 このチェック機能すごい便利ですね!!設定ありがとうございます☺️

kawadon780 commented 2 years ago

LGTM

tokku5552 commented 2 years ago

これもうマージされてるんでレビューしなくて良いですかね?

karokita commented 2 years ago

@tokku5552 はい、ここはいったんレビュー大丈夫です!

今回やってみて、 UI周りは相互レビューと設定いただいたチェック機能でいけそうな気がしてきました!

なので、複数人レビュアーアサインして、 その中でいけそうな人が受け取ってレビューしてく、みたいな進め方でどうでしょう??

ログイン/API/本棚登録周りは徳田さんにみていただくのが必要かなと思うので、 そちらは重点的にお願いできるとありがたいです!(一番大変なところ...!)