Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fetch15 twitter with base64 images #31

Merged
merged 10 commits into from
Aug 25, 2022

Conversation

iory
Copy link
Collaborator

@iory iory commented Aug 22, 2022

fetchでtweetするときにbase64の画像を読み込んで、画像つきでtweetすることを可能にする変更です。
jsk-ros-pkg#375
こちらのPRがupstreamのものです。

@tkmtnt7000
knorth55/jsk_robot#327 こちらのPRでキッチンデモをやってみたときにtweetが画像つきで表示されるのかを確認してみてください。

@tkmtnt7000
Copy link

ありがとうございます.早速試してみます.

@iory
Copy link
Collaborator Author

iory commented Aug 22, 2022

@tkmtnt7000 Thanks!

tkmtnt7000/jsk_robot#7
I made a change to see it as a THREAD when tweeting.
It would be great if you could check again.

Co-authored-by: Naoto Tsukamoto <naototukka0413@gmail.com>
@tkmtnt7000
Copy link

tkmtnt7000 commented Aug 24, 2022

I think it's time to merge this PR, so could you please add commit if you have something to update and when you have time? @iory

@iory
Copy link
Collaborator Author

iory commented Aug 25, 2022

Thanks. I added commit of upstream jsk-ros-pkg#375

@tkmtnt7000
Copy link

Thank you for updating. It looks good.

@iory
Copy link
Collaborator Author

iory commented Aug 25, 2022

@tkmtnt7000 Thanks for your review! Merge this PR.

@iory iory merged commit 841eef4 into knorth55:fetch15 Aug 25, 2022
@iory iory deleted the fetch15-twitter-with-base64-images branch August 25, 2022 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants