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

No output from txtv #9

Open
fhelmer opened this issue Dec 25, 2022 · 4 comments
Open

No output from txtv #9

fhelmer opened this issue Dec 25, 2022 · 4 comments

Comments

@fhelmer
Copy link
Collaborator

fhelmer commented Dec 25, 2022

Since some time, I no longer get any output from txtv. (e.g $ txtv 100)

Seemes like txtv searches for "div" tag with "class" Content_screenreaderOnly__Gwyfj

If I check in "View source" in Chrome e.g. view-source:https://www.svt.se/text-tv/130 , it's now called

Content_screenreaderOnly__1kao2

When I updated this line in txtv.py it appears to work again (at least on my machine). I tried to push the update to a new branch for a pull request, but got "Permission denied".

@voidcase
Copy link
Owner

Thanks for looking into this! You are getting permission denied because you are not a collaborator of this repo, but since you are showing such initiative I just invited you to become one ;)

@fhelmer
Copy link
Collaborator Author

fhelmer commented Dec 29, 2022

Thanks :) Now it was possible to create a PR and I just noticed that the change was merged so please close this issue.

Also thanks for creating 'txtv'. It was very useful last year when I participated in a sailing across the atlantic. Due to the super slow data speed of Iridium Satellite phones (2.4 kbit/s), it was difficult to use anything other than plain text for the daily news.

@janerikj
Copy link

janerikj commented Jul 29, 2024

SVT has updated the source once again: 1kao2 -> 3Cnkp

Edit: Above change already implemented in txtv. The pip version has the old code though.

@fhelmer
Copy link
Collaborator Author

fhelmer commented Dec 1, 2024

SVT has updated the source once again: 1kao2 -> 3Cnkp

Edit: Above change already implemented in txtv. The pip version has the old code though.

I suppose setup.py needs to be updated with a new version number and then upload a new version to PyPi somehow.

Maybe we can ask @voidcase to do this at some point :)

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