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

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 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

2 participants