fixme icon indicating copy to clipboard operation
fixme copied to clipboard

Added Twitter Hook

Open Muljayan opened this issue 7 years ago • 7 comments

closes #14

Included a twitter hook

image

No tweets are present in the fixmeparser twitterpage

Muljayan avatar Oct 21 '18 17:10 Muljayan

the widget box currently looks too big how about setting a max-height for the widget. I think showing about 2 tweets at a time would be just fine. @Muljayan

himankpathak avatar Oct 22 '18 15:10 himankpathak

@Muljayan nice work : ) could you display only one tweet and try to keep it as close as possible to this deigns https://github.com/ossn/fixme/blob/master/screens/desktop/Mozilla_Fixme_Screen01_Homepage_A.png ?

alexdor avatar Oct 22 '18 15:10 alexdor

@himankpathak thanks for the suggestion :) @alexdor i noticed that the twitter api has a limit to get tweets, on the free tier that is. How about putting a get request to the twitter api on our backend every few minutes or so to fetch the latest tweets and then make the front request from our back end.

Muljayan avatar Oct 22 '18 16:10 Muljayan

@Muljayan that's a good idea, what's the limit for the twitter api?

alexdor avatar Oct 24 '18 16:10 alexdor

image image

It says 900-1500 requests per window. Where one window is 15mins . So i think if we don't have 900-1500 people using the app in a span of 15 mins at once we should be fine. We might also have to consider those working on PRs to further enhance this app. When ever they start the server, refresh the page etc. It'll be taken as a hit right

Muljayan avatar Oct 24 '18 17:10 Muljayan

Is anyone working on this right now?

siddhant1 avatar Feb 28 '19 10:02 siddhant1

As far as I know, no it would be nice to have an implementation which is closer to the designs https://github.com/ossn/fixme/blob/master/screens/desktop/Mozilla_Fixme_Screen01_Homepage_A.png

alexdor avatar Feb 28 '19 12:02 alexdor