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

Handle share links #16

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

Conversation

half-duplex
Copy link
Member

@half-duplex half-duplex commented Sep 26, 2024

Fixes #8

Test links (from the praw commit, I'm not touching the reddit app):
https://www.reddit.com/r/redditdev/s/WNauetbiNG (submission)
https://www.reddit.com/r/redditdev/s/nGnQE1QkLC (comment)

Example output:

me> https://www.reddit.com/r/redditdev/s/WNauetbiNG
Sopel> [reddit] [PRAW] HTTPS enabled PRAW testing needed ('PRAW' flair) (self.redditdev) | 11 points (84.0%) | 2 comments | Posted by bboe | Created at 2014-09-17 - 06:34:31 UTC | https://redd.it/2gmzqe
me> https://www.reddit.com/r/redditdev/s/nGnQE1QkLC
Sopel> [reddit] Comment by bboe | 1 point | Posted at 2014-09-18 - 03:13:54 UTC | https://reddit.com/comments/2gmzqe/c/cklhv0f | Yes it does. That fix is also in the master branch, we just haven't made a release for it (and we probably won't until 3.0).

sopel_reddit/__init__.py Outdated Show resolved Hide resolved
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.

Figure out how to handle shortened share links
2 participants