-
Notifications
You must be signed in to change notification settings - Fork 2k
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
(Do not merge) displays latest tweet date, closes #321 #457
Conversation
Deploy preview for landscape ready! Built with commit ac92616 |
Deploy preview for landscape ready! Built with commit a0fe5ff |
Deploy preview for landscape ready! Built with commit f176177 |
And please add a sort by last tweet. |
tools/twitter.js
Outdated
try { | ||
source = require('js-yaml').safeLoad(require('fs').readFileSync('processed_landscape.yml')); | ||
} catch(_ex) { | ||
console.info('Cannot extract image entries from the processed_landscape.yml'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be Cannot extract Twitter entries...
Twitter address fixes: 93f668a You will need to run again in an hour to pickup changes on Crunchbase. |
Ok, I'll run this again. |
And will add a requirement that a twitter is valid |
@dankohn , here is a list of issues with twitter in our data:
|
Beside that, this is ready to merge |
Signed-off-by: Dan Kohn <dan@dankohn.com>
Signed-off-by: Dan Kohn <dan@dankohn.com>
* wip: latest tweet date * taking a proper twitter account * support a proper twitter account * handle a case when there are no tweets at all * do not repeat twitter entries * calculating an effective twitter * show latest tweet together with a twitter name when on a mobile device * allow to sort by a latest tweet date * fix a twitter link * new data after yarn update * OpenContrail twitter fix * rename effectiveTwitter to actualTwitter * do not allow entries with an empty or invalid twitter * Merge master and yarn fetch Signed-off-by: Dan Kohn <dan@dankohn.com>
* wip: latest tweet date * taking a proper twitter account * support a proper twitter account * handle a case when there are no tweets at all * do not repeat twitter entries * calculating an effective twitter * show latest tweet together with a twitter name when on a mobile device * allow to sort by a latest tweet date * fix a twitter link * new data after yarn update * OpenContrail twitter fix * rename effectiveTwitter to actualTwitter * do not allow entries with an empty or invalid twitter * Merge master and yarn fetch Signed-off-by: Dan Kohn <dan@dankohn.com> Former-commit-id: e2e0d5e
* wip: latest tweet date * taking a proper twitter account * support a proper twitter account * handle a case when there are no tweets at all * do not repeat twitter entries * calculating an effective twitter * show latest tweet together with a twitter name when on a mobile device * allow to sort by a latest tweet date * fix a twitter link * new data after yarn update * OpenContrail twitter fix * rename effectiveTwitter to actualTwitter * do not allow entries with an empty or invalid twitter * Merge master and yarn fetch Signed-off-by: Dan Kohn <dan@dankohn.com> Former-commit-id: e2e0d5e
* wip: latest tweet date * taking a proper twitter account * support a proper twitter account * handle a case when there are no tweets at all * do not repeat twitter entries * calculating an effective twitter * show latest tweet together with a twitter name when on a mobile device * allow to sort by a latest tweet date * fix a twitter link * new data after yarn update * OpenContrail twitter fix * rename effectiveTwitter to actualTwitter * do not allow entries with an empty or invalid twitter * Merge master and yarn fetch Signed-off-by: Dan Kohn <dan@dankohn.com>
* wip: latest tweet date * taking a proper twitter account * support a proper twitter account * handle a case when there are no tweets at all * do not repeat twitter entries * calculating an effective twitter * show latest tweet together with a twitter name when on a mobile device * allow to sort by a latest tweet date * fix a twitter link * new data after yarn update * OpenContrail twitter fix * rename effectiveTwitter to actualTwitter * do not allow entries with an empty or invalid twitter * Merge master and yarn fetch Signed-off-by: Dan Kohn <dan@dankohn.com>
* wip: latest tweet date * taking a proper twitter account * support a proper twitter account * handle a case when there are no tweets at all * do not repeat twitter entries * calculating an effective twitter * show latest tweet together with a twitter name when on a mobile device * allow to sort by a latest tweet date * fix a twitter link * new data after yarn update * OpenContrail twitter fix * rename effectiveTwitter to actualTwitter * do not allow entries with an empty or invalid twitter * Merge master and yarn fetch Signed-off-by: Dan Kohn <dan@dankohn.com>
Please don't merge this yet, because I have not added fatal checks for a lack of twitter.
And I will not add them until you preview and tell how this looks like or I can not build and can not show anything