fix:issue515 I want to be able to go to GitHub and Twitter and etc.. …

[kiuchikeisuke]

…of the speaker in SpeakerDetailFragment .

Issue

• close I want to be able to go to GitHub and Twitter and etc.. of the speaker in SpeakerDetailFragment . #515

Overview (Required)

• If click speaker image, show SNS link button on bottomSheet.

Links

Screenshot

after

[takahirom]

Thank you for useful feature!

[takahirom]

Please resolve conflict 🙏

[kiuchikeisuke]

すいません、ちょっと日本語で。。。
コンフリクト解消の時って以下の手順であってますか？

1:fork先のレポジトリにfork元の変更を全て持ってくる
2:fork先のレポジトリのbranch(issue515_showsnslink)にfork元のmasterの変更をマージする
3:コンフリクトを解消してコミット
4:プッシュ→PR作成

気になったのは３の時のコミットファイルがmasterのこれまでの差分全部になっていたので、このままコミット→プッシュ→PRしたら、いらぬコミットログ残しそうな気がしてしまい。。

[jmatsu]

@kiuchikeisuke

合ってます。一応コマンドだと以下になります。

# origin に @kiuchikeisuke さんのリポジトリがあるとして、丁寧にやると

git remote add droidkaigi git@github.com:DroidKaigi/conference-app-2018.git
git fetch droidkaigi master
git checkout issue515_showSNSlink
git merge --no-ff droidkaigi/master
# コンフリクト解消
git commit
# 適宜編集、保存してエディタを閉じる
git push origin issue515_showSNSlink

気になったのは３の時のコミットファイルがmasterのこれまでの差分全部になっていたので

コミット後のエディタを閉じたとき(or コミットメッセージ下部のコメント)に、更新されたファイル一覧が出ることを指してますか？だとしたら問題ないです。
git diff --name-only droidkaigi/master といったコマンドを叩けば、github webのFile Changedに表示されるファイル一覧が確認できるかと思います。

[kiuchikeisuke]

@jmatsu ありがとうございます！問題なくPushできました！！ 😄 😄

[takahirom]

Almost LGTM 👍
If you can, please cherry-pick this 🙏
I think dialogBinding.speaker?.twitterUrl is not good. So I refactor that
519bc4a

[takahirom]

Thanks 👍 LGTM

[takahirom]

Thank you for very useful feature 🥇

[kiuchikeisuke]

Thank you for refactoring!! I cherry-picked it!! 😄