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

manipulate stations.yml with web-service and som extentions on landing page #117

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

Conversation

ThHanika
Copy link

I hop it helps, here the website

YCast webpage

@Essoloani
Copy link

only ip-adress of your raspbery?

It's better with the IP. :-D

@Essoloani
Copy link

Only the first category of my favorites is displayed and the others are deleted by adding a station.
And again the UFT-8 problem, all the existing accented characters are modified.

@ThHanika
Copy link
Author

ThHanika commented Feb 13, 2022

I hope you have a backup, and... the web is still evolving. Ahh, I see you didn't select a category in the bookmark header... then 'others' will automatically be set as the bookmark category! In this case, all other bookmark items with a different category are hidden, but not deleted.
If you clear the Category field, all bookmarks will reappear.
(I was thinking about how it is possible to be able to specify arbitrary names for further categories. Of course, this may be a bit confusing)

@ThHanika
Copy link
Author

ThHanika commented Feb 13, 2022

And again the UFT-8 problem, all the existing accented characters are modified.

Even if I repeat myself, the é is not a legal character in a *.yaml file. It is encoded. The stations.yml is just a location. I tried it: The character is displayed correctly on the AVR amplifier.

BTW: If you modified the stations.yml and set the é in the stations.yml file, it es possible, the python cannot decode it.
(OffTopic: "Chérie Fm" is a nice station... 👍

@Essoloani
Copy link

The two attached files show the modifications of stations.yml after using the web service.
Listes de stations.txt
Listes de stations2.txt
.

@ThHanika
Copy link
Author

Aaah thank you for the file, there is a bug with empty favicon-link. It stops on error-exception. Fix in process on the website

@Essoloani
Copy link

Perfect for me!

@mnowok
Copy link

mnowok commented Jan 2, 2023

Could you create a Docker image with your fork? I use ycast docker on Qnap with success, but it looks like your commits would not be merged into original project anytime...

@ThHanika
Copy link
Author

ThHanika commented Jan 2, 2023

Sorry, I have no knowhow about docker.

@mnowok
Copy link

mnowok commented Jan 2, 2023

Me too ☹️ Maybe I'll ask some ycast docker maintainer to do it. There are at least a few Dockers with ycast so i think it is just a matter of changing ycast repository in docker configuration.

André van Dijk and others added 9 commits January 18, 2023 10:38
…automatically, use SIHUP to trigger. API to query/modify filters/limits.
Filter/limits control API
Documented format in filter.yml.example.
Bumped the version number to 1.3.0.
Added example api calls.
@Essoloani
Copy link

The filter.yml file is not automatically created in my working directory, is this normal?
I only have the file /var/www/ycast/.ycast/recently.yml

@ThHanika
Copy link
Author

This is normal.
a default Dictionary is created in memory, and up to now, there is nothing to save.
But there is an newcommer, fit in nextcloud developement. He makes global filters editable by web-frontend (working in prozess)

@ThHanika
Copy link
Author

Could you create a Docker image with your fork? I use ycast docker on Qnap with success, but it looks like your commits would not be merged into original project anytime...

@ThHanika/YCast is now a docker-file made

ThHanika and others added 7 commits February 24, 2023 11:12
Overhaul of filter/limits
Added  --break-system-packages  to pip3 command lines to fix build error (experienced on Ubuntu and Debian)
Dockerfile gives error on pip3 lines
Add entry_points in setup.py to enable installation via pipx
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.

6 participants