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

[kiuchikeisuke]

…of the speaker in SpeakerDetailFragment .

Issue

• close #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 https://github.com/DroidKaigi/conference-app-2018/commit/519bc4ac613a6ce581058dbd52207f840ce8a9c0

[takahirom]

Thanks 👍 LGTM

[takahirom]

Thank you for very useful feature 🥇

[kiuchikeisuke]

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