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

Skin reverts to english after cheetagenerator runs [BUG] #221

Closed
dystechnic opened this issue Nov 5, 2023 · 5 comments
Closed

Skin reverts to english after cheetagenerator runs [BUG] #221

dystechnic opened this issue Nov 5, 2023 · 5 comments
Labels
bug Something isn't working

Comments

@dystechnic
Copy link

Describe the bug
I use version 3.4.0 of the skin.
I use it with the nl translation and initially all is fine.The skin is in dutch as expected. But after a while the skin reverts back to en.

To Reproduce
Steps to reproduce the behavior:

  1. enable dutch in weewx.conf (lang = nl) and in skin.conf (date_time_locale = nl-NL).
  2. run wee_reports to generate the needed files.The skin is now in the correct language. (dutch)
  3. wait for cheetagenerator to run. I check with 'systemctl status weewx.service'. This runs about every 5 minuts.
  4. refresh the page in your browser
  5. the skin has reverted to english in my case.

Expected behavior
The skin stays in the expected language. Dutch in my case.

Additional context
Version 3.3.0 didnot have this issue. Don't know if this helps or not.

@dystechnic dystechnic added the bug Something isn't working label Nov 5, 2023
@dystechnic
Copy link
Author

Some extra info:
16:41 run wee_reports ==> Skin in dutch

systemctl status weewx.service'

nov 05 16:45:15 weewx weewxd[21904]: weewx[21904] INFO weewx.cheetahgenerator: Generated 8 files for report SeasonsReport in 0.74 seconds
nov 05 16:45:15 weewx weewxd[21904]: weewx[21904] INFO weewx.imagegenerator: Generated 15 images for report SeasonsReport in 0.35 seconds
nov 05 16:45:15 weewx weewxd[21904]: weewx[21904] INFO weewx.reportengine: Copied 0 files to /home/weewx/public_html
nov 05 16:45:15 weewx weewxd[21904]: weewx[21904] INFO weewx.cheetahgenerator: Generated 12 files for report forecast in 0.08 seconds
nov 05 16:45:15 weewx weewxd[21904]: weewx[21904] INFO weewx.reportengine: Copied 0 files to /var/www/html/weewx/forecast

As of this moment the skin is in english.

Daveiano added a commit that referenced this issue Nov 6, 2023
@Daveiano
Copy link
Owner

Daveiano commented Nov 6, 2023

Can you please verify that the file skins/weewx-wdc/lang/nl.conf exists?

I forgot to include it in the "install script" (already fixed, see 091edd0), but I assume the file exists for you because you had copied it earlier.

However, I can reproduce this behaviour, but only on my local test machine, when I change the language on my production machine, it stays the correct one and does not change. So I assume its a config mistake somewhere, I will do some more research...

@Daveiano
Copy link
Owner

Daveiano commented Nov 6, 2023

I think I figured it out, after looking in my weewx directory, I saw a lot of weewx.conf.backup files:

Screenshot from 2023-11-06 18-56-04

After removing these with

sudo rm weewx.conf.2022*
sudo rm weewx.conf.2023*

and restarting weewx, the language does not change anymore. Can you please have a look if it's the same problem for you?

@dystechnic
Copy link
Author

Yes, removing the old log files did the trick.
I don't understand why, but thank you for finding this solution.

@Daveiano
Copy link
Owner

Daveiano commented Nov 7, 2023

I don't understand why, but thank you for finding this solution.

Me neither but this is not related to the skin. The language is handled by WeeWX itself. I tested this and it is the same for the default Seasons Skin. You could post to https://groups.google.com/g/weewx-user/ if it bothers you.

@Daveiano Daveiano closed this as completed Nov 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants