docker-ttrss icon indicating copy to clipboard operation
docker-ttrss copied to clipboard

Reseting by default theme at staturp disabled using an environment variable

Open valvin1 opened this issue 6 years ago • 3 comments

Hello,

I was looking if the feedlish will be integrated and I can see it is already done :)

I've seen there is a new feature which reset to default the theme at startup. I understand this feature after what happened recently.

Would it be possible to make it optional using an environment variable with the feature enabled by default ?

I'm not sure users will appreciate to reselect their themes each time I restart the container.

Thank you

valvin1 avatar Dec 22 '18 17:12 valvin1

I'll have a look into this, but the default will be still having the theme reset to the default one. Broken theming simply has bitten me too many times in the past.

x86dev avatar Jul 25 '19 21:07 x86dev

Please have a try with https://github.com/x86dev/docker-ttrss/commit/8565deba18eb543c0170df5a7100026ecca7c3eb

x86dev avatar Jul 25 '19 21:07 x86dev

Hello @x86dev, I only saw your answer and tries it. Unfortunately it doesn't work. I think the issue is located in root/srv/ttrss-configure-db.php

if (env('TTRSS_THEME_RESET', '1')) {

if variable is set to "0" it returns TRUE if set to "" it sets to "1". So it doesn't matter the value it always resetting theme.

valvin1 avatar Sep 24 '19 19:09 valvin1