Skip to content
This repository has been archived by the owner on Sep 23, 2022. It is now read-only.

Some possible improvements #3

Open
stof opened this issue Aug 21, 2011 · 2 comments
Open

Some possible improvements #3

stof opened this issue Aug 21, 2011 · 2 comments

Comments

@stof
Copy link
Contributor

stof commented Aug 21, 2011

I see some possible improvements in the bundle:

  • I confirmed the issue with non-escaped characters in HTML. Use this tweet to confirm (and imagine what could be done by writing an HTML entity in the tweet...)
  • the fetcher allows to change the limit (defaulting to 10) but the controller always rely on the default value of the fetcher
  • the username in the tweets when fetching the tweets for the username Stof70 will be Christophe Coevoet. Is it intended or simply an error when using the data sent by twitter ?
  • The twitter api allows to ignore replies instead of filtering them after getting the results

Also, the bundle forces to ignore replies. Shouldn't it be configurable ?

@stof
Copy link
Contributor Author

stof commented Aug 21, 2011

Also, replacing the links could be changed to use Twitter entities instead of parsing them again (not sure how feasible it is once we do proper escaping)

@maxvanceffer
Copy link

It will be great also fetch users avatars

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants