Skip to content
This repository was archived by the owner on Mar 19, 2025. It is now read-only.

Reseting by default theme at staturp disabled using an environment variable #33

Open
valvin1 opened this issue Dec 22, 2018 · 3 comments

Comments

@valvin1
Copy link

valvin1 commented Dec 22, 2018

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

@x86dev
Copy link
Owner

x86dev commented Jul 25, 2019

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
Copy link
Owner

x86dev commented Jul 25, 2019

Please have a try with 8565deb

@valvin1
Copy link
Author

valvin1 commented Sep 24, 2019

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.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants