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

Twitter enable/disable_notifications calls are deprecated #11

Merged
merged 4 commits into from Jan 30, 2015

Conversation

Projects
None yet
2 participants
@pwr22

pwr22 commented Jan 30, 2015

Switches notify to use 1.1 calls, adds the ability to set retweet preferences, doesn't update unless necessary and fixes an old bug

pwr22 added some commits Jan 30, 2015

use update_friendships instead of notify, it is deprecated
Also corects documention, notify uses screen_name, not ID
Add retweets command to toggle timeline display of retweets per user
Refactored to use common update method for notify and retweets

@pwr22 pwr22 changed the title from Notify deprecated to Twitter enable/disable_notifications calls are deprecated Jan 30, 2015

@semifor

View changes

Show outdated Hide outdated lib/POE/Component/Server/Twirc.pm Outdated
@semifor

This comment has been minimized.

Show comment
Hide comment
@semifor

semifor Jan 30, 2015

Owner

Aside from a single suggested change, LGTM!

Owner

semifor commented Jan 30, 2015

Aside from a single suggested change, LGTM!

pwr22 added some commits Jan 30, 2015

Fix an old bug with notify on / off
It used to accept notify onanycharshere and notify anycharshereoff
@pwr22

This comment has been minimized.

Show comment
Hide comment
@pwr22

pwr22 Jan 30, 2015

Rebased to use hashref vs hash

pwr22 commented Jan 30, 2015

Rebased to use hashref vs hash

semifor pushed a commit that referenced this pull request Jan 30, 2015

Marc Mims
Merge pull request #11 from pwr22/notify-deprecated
Twitter enable/disable_notifications calls are deprecated

@semifor semifor merged commit 7ed1f92 into semifor:master Jan 30, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment