pj-picbook / picbook

9 stars 0 forks source link

Closed karokita closed 2 years ago

karokita commented 2 years ago

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

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

生年月日関連を消した

スクリーンショット 2022-09-27 20 36 07

tokku5552 commented 2 years ago

あ、すみません、LGTMしちゃいましたが生年月日の登録がない場合、年齢を取得しようとしてエラーにはなりませんかね？

karokita commented 2 years ago

@tokku5552 あ、そうだったーーーーー！お伝え漏れてたのですが、早く対応して申請だそうというところで下記のリリースブランチでエラー対応してました🙇‍♀️

auth_repository部分を変更することで、エラーが解消していますので、上記URLご確認いただけるとありがたいです！

(このブランチ混乱しそうなので削除が良いですかね &今回みたいな運用変えなきゃかなと思うのでやり方アドバイスいただきたいです！）

tokku5552 commented 2 years ago

@karokita そもそもこのブランチの内容はreleaseブランチで対応しているよ　という意味ですかね？このブランチ消しちゃって問題なければ、クローズしちゃった方が混乱しなさそうですね 🤔

今回みたいな運用変えなきゃかなと思うのでやり方アドバイスいただきたいです！

そうですね、リリース作業を1人で対応する場合は、すべてリリースブランチでリジェクト食らった部分の修正までしちゃって良いかなと思います！結構修正膨れそうであればリリースブランチから派生ブランチ切って、リリースブランチにPR作ることで変更部分をわかりやすくするとかそんな感じでしょうか 🤔

tokku5552 commented 2 years ago

@karokita こちらの変更は #166 で対応していると思うのでクローズします。問題あればReopenしていただければと思います。