weatherforecast showing only two days.

ArthurOff

@sdetweil haha, got it!

DavidCarreira

@oemel09 Thanks, your fix is working fine!

FatTony123

@oemel09… yep, just can confirm. This fix works like charm ;-)

Wilfried

@oemel09 Perfect. Thanks for the workaround.

Hindenberg

@oemel09 thx for your support. Different to many others, this fix doesn’t work for me. I’m running MM in a docker container and am not able to keep line 297 of weatherforecast.js edited. It always jumps back into its origin version:

params += "&cnt=" + (this.config.maxNumberOfDays < 1 || this.config.maxNumberOfDays > 17 ? 7 : this.config.maxNumberOfDays);

I tried in /mounts/modules/default/weatherforecast/ and also inside the container. I also set the container environment: MM_OVERRIDE_CSS: "false", but no way, after restarting the container, it is always showing the “old” values above
Any hints?

amarand

@oemel09

Patch worked for me, thank you!

karsten13

@Hindenberg

If you are using this setup https://gitlab.com/khassel/magicmirror#default-modules you need also set MM_OVERRIDE_DEFAULT_MODULES: "false"

Hindenberg

@karsten13 was pretty sure I did this. Works perfect, thanks a lot.

kayakbabe

@castletonroad I have had the same eperience too.

zxjason

@oemel09 Thanks, your fix is working fine!