🐛 ⚡️ "createdAt" -> "created_at", remove intermediary file save #2

TTtie · 2020-08-30T18:15:52Z

This PR:

fixes issues with looking up the latest tweets of an user (the "createdAt" field is snakecase according to Twitter dev docs)
improves speed of posting to Instagram by passing the buffer from the underlying browser library straight to the API.

Ref: https://developer.twitter.com/en/docs/twitter-api/v1/data-dictionary/overview/tweet-object#tweet-dictionary

node-image-to-html allows you to get output straight from puppeteer, therefore no need for intermediary files lying on your FS

vercel · 2020-08-30T18:15:56Z

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/nestdotland/tweetshot/i2qltvesi
✅ Preview: Failed

[update for 4b0a66b failed]

t8

LGTM

Thanks for fixing this. I'm not sure what our future plans with this repo are yet, but this is a step in a nice direction!

CC @maximousblk, @divy-work

littledivy

LGTM! I'm not sure about the project's future too, if we can get this working that'd be awesome!

TTtie · 2020-09-01T04:17:29Z

Might as well fix the scheduled CI during today, as that seems to be a bit malformed as well, please don't merge yet

TTtie added 2 commits August 30, 2020 20:10

🐛 (twitter): snake_case "createdAt" -> "created_at"

eff2857

Ref: https://developer.twitter.com/en/docs/twitter-api/v1/data-dictionary/overview/tweet-object#tweet-dictionary

⚡️ (instagram): Remove intermediate file saving

46dbc17

node-image-to-html allows you to get output straight from puppeteer, therefore no need for intermediary files lying on your FS

vercel bot had a problem deploying to Preview August 30, 2020 18:15 Failure

🎨 put the newline back

22a011b

vercel bot had a problem deploying to Preview August 30, 2020 18:18 Failure

t8 approved these changes Aug 31, 2020

View reviewed changes

littledivy approved these changes Sep 1, 2020

View reviewed changes

💚 Fix workflow scheduling

4b0a66b

vercel bot had a problem deploying to Preview September 1, 2020 14:03 Failure

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🐛 ⚡️ "createdAt" -> "created_at", remove intermediary file save #2

🐛 ⚡️ "createdAt" -> "created_at", remove intermediary file save #2

TTtie commented Aug 30, 2020

vercel bot commented Aug 30, 2020 •

edited

Loading

t8 left a comment

littledivy left a comment

TTtie commented Sep 1, 2020

🐛 ⚡️ "createdAt" -> "created_at", remove intermediary file save #2

Are you sure you want to change the base?

🐛 ⚡️ "createdAt" -> "created_at", remove intermediary file save #2

Conversation

TTtie commented Aug 30, 2020

vercel bot commented Aug 30, 2020 • edited Loading

t8 left a comment

Choose a reason for hiding this comment

littledivy left a comment

Choose a reason for hiding this comment

TTtie commented Sep 1, 2020

vercel bot commented Aug 30, 2020 •

edited

Loading