pj-picbook / picbook

9 stars 0 forks source link

Closed karokita closed 2 years ago

karokita commented 2 years ago

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

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

karokita commented 2 years ago

コメントありがとうございます！いただいた2点についてなのですが、私がproviderを触ったことがなく調べるところからになって時間がかかりそうです…

マストじゃない場合はいったん現状でissueクローズさせていただいて別issueで追って対応、マストな場合はいったんこのissue自体をあと回す対応（場合によっては、今回から落とす対応）でいければと思うのですが、どうでしょうか？

（前回ミーティングでパッケージを入れて基本的なとこ動かすとこまでで一旦プルリクを出すという認識だったので、エラー対応の洗い出しとアプリリリース周り調べるの優先してやりたいなという気持ちです！）

tokku5552 commented 2 years ago

@karokita Provider化は私が別のチケットで適当にやっておくので、取り下げました！改めて対応お願いします！

karokita commented 2 years ago

@tokku5552 provider化の件、ありがとうございます🙏🙏🙏 flutter経験値低すぎてすみません！徳田さんのコードを見ながら勉強できたらと思います🙇‍♀️

karokita commented 2 years ago

@tokku5552 finalで定義する件ですが、下記記事を参考にして反映しました。 https://qiita.com/kasa_le/items/ac8de49615acf78129c0

認識にずれないかご確認おねがいします！

tokku5552 commented 2 years ago

@karokita ちなみにブランチが大文字始まりになっており、またプルできなくなったのですが、以降全部小文字での作成お願いできますか？あとスネークケースです(ハイフンじゃなくてアンダーバー)

karokita commented 2 years ago

@tokku5552 小文字とスネークケースで統一の旨、了解です！以降そのようにブランチ作成します。