search

yasasii-team / Stepippo-iOS

初めての一歩を応援するアプリのiOS版
https://github.com/yasasii-team/Stepippo-iOS.git
21 stars 5 forks source link

TextFieldを削除し、Buttonに変更しました #130

Closed eyener3 closed 5 years ago

eyener3 commented 5 years ago

fixes #123 fixes #87 (#の後にcloseしたいissueの番号を記述してください)

Summary(要約)

Other Information(他の情報)

Tested(テストしたこと)

よろしくお願いいたしますm(._.)m

riscait commented 5 years ago

123 はこのPRで完了しないと思われます🙆‍♂️

riscait commented 5 years ago

PRをいくつかに分けていますが、全部同じファイル（Storyboard）を編集しているのでコンフリクトしそうですね🤔 関連する変更ですし、#129 での修正で良かったかもですね👌 （最初はなかなか判断が難しいですよね…！）

コンフリクトの解消になって良いかもしれないです💪

eyener3 commented 5 years ago

ありがとうございます^-^ ちょっと判断に迷ってしまったので、 先にお聞きしたらよかったなと思いました^-^。

他の作業の時に、別のブランチでやると 作業が反映されていないまま進んでいたので あまり良い予感はしませんでした..^-^。

レビューありがとうございました🙇‍♀️

eyener3 commented 5 years ago

LGTMですが、PRを分けすぎるとコンフリクトが発生したり、レビューしにくくなるので避けた方が良いと思います。 1つのタスクを1PRで抑える様なイメージです。 今回の様にPRの修正であれば同じブランチで作業すべきでした。

Kokiさん、ありがとうございます！🙇‍♀️ 今後、気をつけていこうと思います^0^！！