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.

Sign up for GitHub

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

Jump to bottom

Fixing Crash on Sponsored posts #25

Closed
mohland opened this issue Jan 30, 2014 · 2 comments
Closed

Fixing Crash on Sponsored posts #25

mohland opened this issue Jan 30, 2014 · 2 comments

Comments

@mohland
Copy link
Contributor

mohland commented Jan 30, 2014

fyi, tips in promoted posts will crash the bot, since they're not exactly "real" reddit posts.

I just ended up ignoring them with dogetipbot:

mohland@d2fb961
@vindimy
Copy link
Owner

vindimy commented Jan 30, 2014

That's a good suggestion, but I'd put this check earlier in the loop, like in check_inbox() function, before it does any regex matching.

@vindimy
Copy link
Owner

vindimy commented Feb 11, 2014

Looks like b6afdf5 fixes this as a side effect.

@vindimy vindimy closed this as completed Feb 11, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@vindimy @mohland