Skip to content
This repository has been archived by the owner on Feb 8, 2018. It is now read-only.

Video, etc. #4671

Merged
merged 7 commits into from
Oct 6, 2017
Merged

Video, etc. #4671

merged 7 commits into from
Oct 6, 2017

Conversation

chadwhitacre
Copy link
Contributor

@chadwhitacre chadwhitacre commented Oct 4, 2017

Part of gratipay/inside.gratipay.com#1160. 💃

Todo

  • change base back to master

@chadwhitacre chadwhitacre changed the base branch from master to promo-logo October 4, 2017 21:26
@chadwhitacre
Copy link
Contributor Author

Started looking at pulling updates from Medium.

https://gratipay.news/feed
https://stackoverflow.com/questions/10943544/how-to-parse-an-rss-feed-using-javascript

No CORS so we'll have to do it server-side, which means either doing it manually or caching so we don't slow down page load too much.

@chadwhitacre
Copy link
Contributor Author

Actually though let's not block this on #4656. It's too cool let's get it deployed! 💃

cc: @mattbk @JessaWitzel et al.

@chadwhitacre
Copy link
Contributor Author

chadwhitacre commented Oct 5, 2017

Pause! @ChrisECJohns is looking at fixing up the bit where I say "$250" instead of "$250 per developer." slack

@mattbk mattbk self-requested a review October 5, 2017 15:34
@chadwhitacre
Copy link
Contributor Author

Rendering one more once. slack

@chadwhitacre
Copy link
Contributor Author

Planning to merge per @mattbk's approval once right video is in place ...

@chadwhitacre
Copy link
Contributor Author

Rebased and switched to final video. Was 2e5eac7.

@chadwhitacre
Copy link
Contributor Author

Merge approval because @mattbk is traveling: slack.

@chadwhitacre chadwhitacre merged commit 935e5ce into master Oct 6, 2017
@chadwhitacre chadwhitacre deleted the video-etc branch October 6, 2017 21:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants