search

TheScientist / ttrss-pwa

Progressive Web App for Tiny Tiny RSS
Apache License 2.0
28 stars 2 forks source link

[fr] configuration file? (language and server fields) #54

Open user9931 opened 1 year ago

user9931 commented 1 year ago

sorry if its obvious.. but i cant find config file... want to specify language and server address (and never change it agait)

by the way, dont know if its better to create multiple issues.. [FR] language detection from browser instead of specifying manyally ( or fallback to english if browser's language translation not available)

when using the app in /pwa subfolder of ttrss.domain.com where / is ttrss itself, option to specify root directory will be very helpful

also, didnt realized that in case of subfolder, there's no need to specify server name; also noticed that if we specify server address without http(s):// its trying to reash https://ttrss.domain.com/pwa/ttrss.domain.com/api and it only can be discovered in console, ui just show "login failed"

TheScientist commented 1 year ago

Hi, thanks for your interest in this app. To your questions:

sorry if its obvious.. but i cant find config file... want to specify language and server address (and never change it agait)

This is not planned as the app should act as a client for any tt-rss installation. of course it would make sense if you bundle it with your own tt-rss installation. I'll think of reading defaults or initial values from a file

[FR] language detection from browser instead of specifying manyally ( or fallback to english if browser's language translation not available)

I'll implement language detection, but there is currently no translation for french: please have a look here, if you'd like to propose a PR https://github.com/TheScientist/ttrss-pwa/tree/master/src/assets/i18n

also, didnt realized that in case of subfolder, there's no need to specify server name; also noticed that if we specify server address without http(s):// its trying to reach https://ttrss.domain.com/pwa/ttrss.domain.com/api and it only can be discovered in console, ui just show "login failed"

that's indeed a bug. The idea was only to add the protocol and api/ suffix if missing

I'll try to look into the issues, but currently don't spend a lot of time on the project so please be patient :)