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

[BUG] Theme is not correctly saved and reset at each page refresh #774

Closed
4 tasks done
Antiz96 opened this issue Jun 29, 2022 · 5 comments
Closed
4 tasks done

[BUG] Theme is not correctly saved and reset at each page refresh #774

Antiz96 opened this issue Jun 29, 2022 · 5 comments
Assignees
Labels
🐛 Bug [ISSUE] Ticket describing something that isn't working 📌 Keep Open [ISSUE][PR] Prevent auto-closing ❕ Medium Priority [ISSUE] Needs to be dealt with at some point

Comments

@Antiz96
Copy link

Antiz96 commented Jun 29, 2022

Environment

Self-Hosted (Docker)

Version

2.1.0

Describe the problem

Hi,

Context

I'm using dashy in a docker container on a raspberry Pi 4 (using the latest image).

Run command :

sudo docker run -d -p 8080:80 -v /opt/dashy/conf.yml:/app/public/conf.yml --name dashy --restart=always lissy93/dashy:latest

My conf.yml file is visible here

The issue

Since a few days, I have an issue that (I think) is happening since the latest update to the docker image (V2.1.0).
More on that in the additional info field.

What is not working

As you can see, I've set the "Adventure" theme is my conf.yml file (line 11). However, when I launch dashy through my web browser, it shows the default theme (even tho the theme is indeed automatically set to "Adventure" by my configuration file):

Bug_Dashy_1

Re-selecting the "Adventure" theme manually from the "theme" drop down list makes it appears correctly.
However, refreshing/reloading the page makes Dashy fall back down to the default theme (while still having the "Adventure" theme selected, just like on the above screenshot).

-!- It worth mentioning that I initially had this issue with the "Adventure" theme as this is the one I'm using, but I've tried to set other ones in my config file for testing purposes and I had the exact same behavior. According to that, I guess that this issue goes with all themes, not specifically the "Adventure" one. I'll continue to use the "Adventure" theme as an example anyway but it could have been another one -!-

Step to reproduce

Set the "Adventure" theme in the conf.yml file with the following snippet:

appConfig:
  theme: Adventure

Run the Dashy's latest docker image and mount the configuration file as "/app/public/conf.yml":

sudo docker run -d -p 8080:80 -v /opt/dashy/conf.yml:/app/public/conf.yml --name dashy --restart=always lissy93/dashy:latest

Check the Dashy's web page.
The "Adventure" theme is indeed selected but the default theme is shown instead. Re-selecting it manually from the drop down menu makes it appears correctly but refreshing/reloading the page makes it fall back to the default theme, even tho "Adventure" is still selected (at least in my case 😄).

Expected result

Having the "Adventure" theme correctly shown when launching and reloading/refreshing Dashy's web page without having to manually re-select it.

Thank you

Thanks a lot for your awesome work ! Dashy's amazing 😄

Additional info

  • I said I think this issue is happening since the latest docker image (V2.1.0) because I think there's 2 separate parts of this issue:
    Indeed, I'm quite sure that I had the bug showing the default theme instead of the one I chose in my config file (even tho it is selected in the drop down menu) before the release of the latest docker image (V2.1.0).
    However, at that time, I had to re-select my theme manually only one time after a docker run command (so either after the initial launch of Dashy or after an update). I didn't have to redo that until the next docker run command (meaning until the next update).
    Having to re-select my theme manually each time the page is refreshed/reload is however an issue introduced since the latest docker image (V2.1.0) for me.

  • As I said earlier, I could reproduce this behavior using other themes than "Adventure" in my config file. So it is not related to that particular theme I guess.

  • As I said earlier as well, thanks a lot for your awesome work ! 😄

Please tick the boxes

@Antiz96 Antiz96 added the 🐛 Bug [ISSUE] Ticket describing something that isn't working label Jun 29, 2022
@Lissy93
Copy link
Owner

Lissy93 commented Jun 30, 2022

Hi @Antiz96
Sorry about that, I can confirm this is a bug, it seems to be caused by #752

Thanks for the detailed report, that's really helpful.
I'll work on it this afternoon, and update you when fixed :)

/link-dup #773

@Lissy93 Lissy93 added the ‼️ High Priority [ISSUE] An issue or PR that needs to be dealt with urgently label Jun 30, 2022
@Antiz96
Copy link
Author

Antiz96 commented Jun 30, 2022

Hi @Lissy93,

Thanks for you answer !
I'm sorry, I checked the "issues" but not the "discussions" while searching if this issue has already been raised...

Anyway, I can confirm the workaround raised in #773 !
Setting the theme to "adventure" instead of "Adventure" in the configuration fixed the issue !

Hope it helps ! :)

@liss-bot liss-bot added the 👤 Awaiting Maintainer Response [ISSUE] Response from repo author is pending label Jun 30, 2022
@Lissy93 Lissy93 added ❕ Medium Priority [ISSUE] Needs to be dealt with at some point and removed ‼️ High Priority [ISSUE] An issue or PR that needs to be dealt with urgently labels Jun 30, 2022
Lissy93 added a commit that referenced this issue Jun 30, 2022
@liss-bot
Copy link
Collaborator

This issue has gone 6 weeks without an update. To keep the ticket open, please indicate that it is still relevant in a comment below. Otherwise it will be closed in 5 working days.

@liss-bot liss-bot added ⚰️ Stale [ISSUE] [PR] No activity for over 1 month and removed 👤 Awaiting Maintainer Response [ISSUE] Response from repo author is pending labels Jul 31, 2022
@Lissy93 Lissy93 added 📌 Keep Open [ISSUE][PR] Prevent auto-closing and removed ⚰️ Stale [ISSUE] [PR] No activity for over 1 month labels Jul 31, 2022
@Lissy93 Lissy93 closed this as completed Feb 11, 2024
@maks2199

This comment has been minimized.

@CrazyWolf13
Copy link
Collaborator

Hi @maks2199

Please look through the open issues first:

Here the related ticket:
#1604

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bug [ISSUE] Ticket describing something that isn't working 📌 Keep Open [ISSUE][PR] Prevent auto-closing ❕ Medium Priority [ISSUE] Needs to be dealt with at some point
Projects
None yet
Development

No branches or pull requests

5 participants