Skip to content
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

Add in sitestreams handler for quoted tweets #233

Merged
merged 5 commits into from
Aug 11, 2018

Conversation

miketempleman
Copy link
Contributor

No description provided.

@eliasisrael
Copy link
Contributor

These changes permit site streams handlers to manage quoted tweets, and add in handlers for the mute and unmute events.

@eliasisrael
Copy link
Contributor

Oh, darn it. I obviously did not intend to include the pom.xml changes in this pull request. I'll repair that.

@miketempleman
Copy link
Contributor Author

I had already requested a pull for this

From: Eli Israel notifications@github.com
Reply-To: yusuke/twitter4j
<reply+00173e9b75882af6d090ca0f4b004e536d7ad52a0780a2ae92cf00000001130539d09
2a169ce08225cd0@reply.github.com>
Date: Saturday, March 19, 2016 at 8:01 AM
To: yusuke/twitter4j twitter4j@noreply.github.com
Cc: Michael Templeman mike@talkingfrog.net
Subject: Re: [twitter4j] Add in sitestreams handler for quoted tweets
(#233)

Oh, darn it. I obviously did not intend to include the pom.xml changes in
this pull request. I'll repair that.


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#233 (comment)

@eliasisrael
Copy link
Contributor

I think it just added my stuff to the end of it.

On Mar 19, 2016, at 12:55 PM, Michael Templeman notifications@github.com wrote:

I had already requested a pull for this

From: Eli Israel notifications@github.com
Reply-To: yusuke/twitter4j
<reply+00173e9b75882af6d090ca0f4b004e536d7ad52a0780a2ae92cf00000001130539d09
2a169ce08225cd0@reply.github.com>
Date: Saturday, March 19, 2016 at 8:01 AM
To: yusuke/twitter4j twitter4j@noreply.github.com
Cc: Michael Templeman mike@talkingfrog.net
Subject: Re: [twitter4j] Add in sitestreams handler for quoted tweets
(#233)

Oh, darn it. I obviously did not intend to include the pom.xml changes in
this pull request. I'll repair that.


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#233 (comment)


You are receiving this because you commented.
Reply to this email directly or view it on GitHub #233 (comment)

@miketempleman
Copy link
Contributor Author

You probably should note that you added the mute events. Otherwise he might
miss it. siteStreams users will want that.

From: Eli Israel notifications@github.com
Reply-To: yusuke/twitter4j
<reply+00173e9bc5408bcd9346dfd6e0843756cffa006a27859d2a92cf00000001130570f59
2a169ce08225cd0@reply.github.com>
Date: Saturday, March 19, 2016 at 11:56 AM
To: yusuke/twitter4j twitter4j@noreply.github.com
Cc: Michael Templeman mike@talkingfrog.net
Subject: Re: [twitter4j] Add in sitestreams handler for quoted tweets
(#233)

I think it just added my stuff to the end of it.

On Mar 19, 2016, at 12:55 PM, Michael Templeman notifications@github.com
wrote:

I had already requested a pull for this

From: Eli Israel notifications@github.com
Reply-To: yusuke/twitter4j
<reply+00173e9b75882af6d090ca0f4b004e536d7ad52a0780a2ae92cf00000001130539d09
2a169ce08225cd0@reply.github.com>
Date: Saturday, March 19, 2016 at 8:01 AM
To: yusuke/twitter4j twitter4j@noreply.github.com
Cc: Michael Templeman mike@talkingfrog.net
Subject: Re: [twitter4j] Add in sitestreams handler for quoted tweets
(#233)

Oh, darn it. I obviously did not intend to include the pom.xml changes in
this pull request. I'll repair that.

?
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#233 (comment)


You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#233 (comment)


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#233 (comment)

@eliasisrael
Copy link
Contributor

I put a comment in to that effect in the pull request.

On Mar 19, 2016, at 1:08 PM, Michael Templeman notifications@github.com wrote:

You probably should note that you added the mute events. Otherwise he might
miss it. siteStreams users will want that.

From: Eli Israel notifications@github.com
Reply-To: yusuke/twitter4j
<reply+00173e9bc5408bcd9346dfd6e0843756cffa006a27859d2a92cf00000001130570f59
2a169ce08225cd0@reply.github.com>
Date: Saturday, March 19, 2016 at 11:56 AM
To: yusuke/twitter4j twitter4j@noreply.github.com
Cc: Michael Templeman mike@talkingfrog.net
Subject: Re: [twitter4j] Add in sitestreams handler for quoted tweets
(#233)

I think it just added my stuff to the end of it.

On Mar 19, 2016, at 12:55 PM, Michael Templeman notifications@github.com
wrote:

I had already requested a pull for this

From: Eli Israel notifications@github.com
Reply-To: yusuke/twitter4j
<reply+00173e9b75882af6d090ca0f4b004e536d7ad52a0780a2ae92cf00000001130539d09
2a169ce08225cd0@reply.github.com>
Date: Saturday, March 19, 2016 at 8:01 AM
To: yusuke/twitter4j twitter4j@noreply.github.com
Cc: Michael Templeman mike@talkingfrog.net
Subject: Re: [twitter4j] Add in sitestreams handler for quoted tweets
(#233)

Oh, darn it. I obviously did not intend to include the pom.xml changes in
this pull request. I'll repair that.

?
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#233 (comment)


You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#233 (comment)


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#233 (comment)


You are receiving this because you commented.
Reply to this email directly or view it on GitHub #233 (comment)

@yusuke yusuke merged commit 9643ca6 into Twitter4J:master Aug 11, 2018
yusuke added a commit that referenced this pull request Aug 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants