question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

[Weather Module] decimalSymbol has no effect

See original GitHub issue

I set the decimalSymbol in the config of the new weather module to ,, but that has no effect. The values still have points. I saw that the PR #1510 should have fix that, but it doesn’t work (anymore?).

I noticed this with a just freshly installed MagicMirror instance.

Or am I doing something wrong in my config?

    {
      module: "weather",
      position: "top_right",
      header: "Wetter in ?????",
      config: {
        appendLocationNameToHeader: false,
        decimalSymbol: ",",
        weatherProvider: "openweathermap",
        type: "forecast",
        location: "?????????",
        locationID: "?????????",
        maxNumberOfDays: 5,
        apiKey: "?????????",
      }
    }

Issue Analytics

  • State:closed
  • Created 2 years ago
  • Comments:6 (2 by maintainers)

github_iconTop GitHub Comments

1reaction
rejascommented, Apr 11, 2021

Nice. So lets see if your PR fixes my tests 😄

1reaction
rejascommented, Apr 11, 2021

Hi @KristjanESPERANTO I dont think you are doing somethign wrong. I took a quick look at the code and it seems to me the decimal point isnt used (yet) in the forecast part (only in the current weather part) of the module.

Maybe you want to write a fix for it? 😉

Read more comments on GitHub >

github_iconTop Results From Across the Web

MagicMirror - Bountysource
I set the decimalSymbol in the config of the new weather module to , , but that has no effect. The values still...
Read more >
Weather Module - MagicMirror² Documentation
This module will be configurable to be used as a current weather view, or to show the forecast. This way the module can...
Read more >
7-Day Forecast 38.56N 123.31W - National Weather Service
Rain, mainly after 4pm. High near 56. Southeast wind 6 to 13 mph. Chance of precipitation is 80%. New precipitation amounts between a...
Read more >
Agilent 4200 TapeStation System System Manual
preinstalled with your 4200 TapeStation system. 6 Operating Instructions ... If your module was shipped in cold weather, leave it in its box...
Read more >
https://www.touchpadprofoundation.org/wp-content/p...
msgstr "" #: includes/helper.php:451 msgid "No sidebars were found" msgstr ... includes/helper.php:558 modules/parallax-effects/module.php:339 msgid "Hue" ...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found