G button#4 - Githubissues

sanaevvv commented 3 years ago

Please review!!

yugo-ibuki commented 3 years ago

Buttonのコンポーネントとして抽象化するのが良さそうですね！ interfaceでpropsを定義してあげると使いやすいコンポーネントになるかと思います！

簡単に書いてみました！ classNameはコピペなので、共通化してもいいかもですね！

スクリーンショット 2021-10-18 22 35 37 スクリーンショット 2021-10-18 22 35 54

sanaevvv commented 3 years ago

修正しました！どうでしょうか？？

yugo-ibuki commented 3 years ago

@sanaevvv 良いと思います！コミットはされてますでしょうか？

index.tsxは今、Header作成で変更されているので、HEADに戻してもらっても大丈夫です！

sanaevvv commented 3 years ago

ヘッダーのコミットしましたが、順番間違ったかもしれません！大丈夫ですかね？

yugo-ibuki commented 3 years ago

@sanaevvv こちらのファイルがdevelopとのコンフリクトを起こしているので、元の状態に戻していただけると解消できると思います！ https://github.com/Qin-Eva/pj_engivia/pull/9/files?short_path=7ae45ad#diff-18e0d4553c97cfc420e938ccafb4e3a688e782fa4512ba4ceae3e2a6f24c1987

Postってページも追加されましたかね？ちょっとこのPRが肥大化されてきて、どこを見ればいいのか分かりにくくなってしまうのでコンフリクトだけ解消したら、一旦マージしようかと思います！

sanaevvv commented 3 years ago

コンフリクト解消しました！

Postは一時的に作りましたのでマージなしで大丈夫です。

yugo-ibuki commented 3 years ago

コンフリクト解消できてないですね…。

Postは一時的に作りましたのでマージなしで大丈夫です。

Postページは僕の方で削除してコミットするって事でしょうか

sanaevvv commented 3 years ago

あれ？すみません！必要なところはローカルにコピーしましたのでpushしたPOSTは削除してコミットしてもらえると助かります！

yugo-ibuki commented 3 years ago

はーい、ひとまずコンフリクト解消とPostページ削除したので、こちらでマージしちゃいますね。一点、タイポかなっていうところもあったのでついでに直しておきました。

sanaevvv commented 3 years ago

ありがとうございます！

Qin-Eva / pj_engivia