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

Detect tweet transcriptions #2

Open
q-- opened this issue Jul 9, 2018 · 4 comments
Open

Detect tweet transcriptions #2

q-- opened this issue Jul 9, 2018 · 4 comments
Assignees

Comments

@q--
Copy link
Collaborator

q-- commented Jul 9, 2018

Doesn't work right now because of an extra   between If and you're. Should probably switch over to a regex (but need to be careful of performance)

@itsthejoker
Copy link
Member

I could have sworn we fixed that months ago. Is it part of the twitter format specifically or in more places?

@q--
Copy link
Collaborator Author

q-- commented Jul 10, 2018

Part of the Twitter format.

too! [If  you'd like more

Might happen in other places as well, but I haven't checked it.

@q-- q-- self-assigned this Jul 10, 2018
@TheLonelyGhost
Copy link
Member

That's actually super weird and should be corrected in the formatting, not necessarily this tool. No need to work around this on your side.

@q--
Copy link
Collaborator Author

q-- commented Jul 14, 2018

I'm no mod at r/transcribersofreddit so if anyone wants to fix it, be my guest.

Might still be worth accounting for in the code because of older posts people will encounter on subreddit/top. Also, IIRC I read somewhere you had some rule that people could choose their own footer text on April 1st, is that 100% arbitrary or does it still have a wiki link or something else I can try to detect.

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

No branches or pull requests

3 participants