Ta23ka98 commented 2 years ago

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

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

対応したこと*

未ログイン画面を作成
- UIをFigma通りに作成
- 一部のテキストを画面遷移可能にした
WebView画面を作成
- webview_flutterを導入
- 新たにagreement_pageディレクトリ/ファイル/ウィジェットを作成
- 仮のURLを設定し、画面遷移可能な状態にした

Ta23ka98 commented 2 years ago

未解決の課題(WebViewの詳細)については、やまさんに別issueを立てていただく予定です。

tokku5552 commented 2 years ago

こちらで依頼があったのでアサインします。

tokku5552 commented 2 years ago

こちらコンフリクトの修正とlintエラーの修正私の方でしておきました！今から見ます！

Ta23ka98 commented 2 years ago

@tokku5552 対応＆pushしました！ここでAndroid Studioからコンフリクトエラーが出たのですが、どう対応すれば良いのでしょうか🤔

確か「マージする必要がある」と出たのでマージしたのですが、画面はこうなっています。具体的に、何をすればいいかよく分かりません🙇

必要ないかもですが、私見として... ・マージされた変更を破棄する・developを最新の状態にする・developの変更を、今の作業ブランチにマージする・ビルドできるようになるまで、コードを修正する・改めてpushする

こういった認識で間違いないでしょうか？お手数ですが、ご指導お願いします🙇

tokku5552 commented 2 years ago

そもそもローカルブランチとリモートブランチを同期しないまま作業してしまったのでそのような状態になってしまったかと思います 🤔 一旦修正をなかったコトにしてもらって、作業の順序としては

tokku5552 commented 2 years ago

@Ta23ka98 すみません、直したところがまた先祖返りしてます、、、リベースしてマージ下とかですかね？これdevelopブランチに対してもやると困るので、リモートブランチとローカルブランチに差分がない状態で基本作業していただけるとありがたいです 🙏 今回も直しておきますね

Ta23ka98 commented 2 years ago

申し訳ありません、おかしいと感じて質問するところでした🙇 変更ありがとうございます！差分がない状態で作業するようにします。

tokku5552 commented 2 years ago

対応ありがとうございました！マージボタンポチッとお願いします！ LGTM