Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

basic twirssi_search implementation #11

Merged
5 commits merged into from
Sep 28, 2010
Merged

basic twirssi_search implementation #11

5 commits merged into from
Sep 28, 2010

Conversation

mmitch
Copy link
Contributor

@mmitch mmitch commented Sep 17, 2010

adds a twirssi_search command to start a one-time search

includes:

  • search is queued via background job and a twirssi_updates is started
  • queued searches are removed upon the first result (non-existing #hashtags will be searched until they show up or you restart irssi)
  • a twitter_search_results setting has been added to set the maximum number of search results
  • parallel runs of the background job should be prevented by the $update_is_running flag

does not include (help/pointers wanted):

  • search results are not remembered and not checked against already seen tweets

@mmitch
Copy link
Contributor Author

mmitch commented Sep 18, 2010

Should I use $username = "$user@$defservice" instead of $username = &normalize_username( $user ) to expand to the current full username?

This pull request was closed.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant