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] Minor Spelling Error on Some Tiles #168

Closed
gary-hammer opened this issue May 23, 2023 · 4 comments
Closed

[BUG] Minor Spelling Error on Some Tiles #168

gary-hammer opened this issue May 23, 2023 · 4 comments
Labels
bug Something isn't working
Milestone

Comments

@gary-hammer
Copy link

Describe the bug
Rain, ET, and Wind Run use Totals instead of Total.
To Reproduce
Steps to reproduce the behavior:

  1. Go to Main Page
  2. See error

Expected behavior
Just Total on these tiles.

@gary-hammer gary-hammer added the bug Something isn't working label May 23, 2023
@Daveiano
Copy link
Owner

Thank you for spotting this! Sorry, I am not a native English speaker...

I assume it should also be "Weekly total" instead of "Weekly totals", "Monthly total" instead of "Monthly totals" and so on?

@Daveiano Daveiano added this to the 3.3.0 milestone May 24, 2023
Daveiano added a commit that referenced this issue May 24, 2023
@gary-hammer
Copy link
Author

That is correct.
As I said, small thing.

@Daveiano
Copy link
Owner

Thanks for spotting this! I changed the translation file for en. This will be included in the next release.

If you want to change it right now, here are the needed changes (in skins/weewx-wdc/lang/en.conf): ffb22b0#diff-2c608082b550dc4abd44b8989b6bcaff80c214e1b60cc653396c7ec61a4e4156

@gary-hammer
Copy link
Author

Thanks, going to able to test the combined rain tile this afternoon as well.

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