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

Updated to work with praw 5.0.1 #35

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

Conversation

jenroper
Copy link

Praw 3.5 is no longer supported, and the script no longer works. I have updated the script to use Praw 5.0.1. Now access to reddit's api requires the user to have a client_id and client_secret that are linked to a redditor's account. Rather than hard-code my own client_id and client_secret, I've added a praw.ini file and updated the readme file with instructions for registering the app with reddit and getting those two values, which need to be placed in the praw.ini file for the app to work.

jenroper and others added 9 commits August 29, 2017 19:33
Adding support for PRAW 5.0 and not hard coding my personal authentication info.
Adding support for praw 5.0
Updated praw version to 5.0.1
formatting
Added better instructions for using this script in a cron job.
fixed spelling of GNOME
@madelinecameron
Copy link

This branch works great by the way! Thank you! :)

I got this working at home this weekend and now to deploy this to my work computer as well.

@jenroper
Copy link
Author

jenroper commented Sep 7, 2017

@dolohow How about now?

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