search

ledger / ledger.github.io

The source code behind https://ledger-cli.org
9 stars 19 forks source link

Tweets for #ledgercli or #plaintextaccounting not shown on index page #55

Closed afh closed 1 year ago

afh commented 1 year ago

Even if there haven't been any tweets with one of the above hashtags or both, I'd expect the timeline to show old tweets. Is it no longer possible to embed a timeline in another webpage? How do we feel about removing this feature from the frontpage, it does not seem to add value in its current state?

/cc @tbm, @peterkeen

afh commented 1 year ago

It seems to be possible to embed a timeline for an account, e.g. https://publish.twitter.com/?dnt=1&maxwidth=320&query=%40LedgerTips&widget=Timeline

but not for a hashtag…

simonmichael commented 1 year ago

IIRC this widget died very long ago and the way Twitter is now I wouldn't even bother. But YMMV.

afh commented 1 year ago

Thanks for the response, @simonmichael, well put. I've removed the Twitter timeline integration in #54.

afh commented 1 year ago

The Twitter timeline should be removed once the latest version is deployed; pending resolution of #46.

afh commented 1 year ago

This change has been deployed.