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

Modul doesnt load #1

Open
wants to merge 47 commits into
base: master
Choose a base branch
from
Open

Modul doesnt load #1

wants to merge 47 commits into from

Conversation

matze6486
Copy link

Hey,
i installed your module (clone + npm install) and i use the sample config with my own api and my own lat/long, but the module displays "load" the whole time.
her is my config. maybe you can fix my problem

{
module: "MMM-OpenWeatherMapForecast",
header: "Weather",
position: "top_right",
classes: "default everyone",
disabled: false,
config: {
apikey: "xxxxxxxxxxxxx",
latitude: "51.78",
longitude: "6.12",
iconset: "4c",
concise: false,
forecastLayout: "table"
}
},

@albertusQuelle
Copy link

For me its also not working !

@sdetweil
Copy link

sdetweil commented Oct 8, 2021

looks like the version check is 2.2.0, but we are only at 2.1.7

think the check should be 2.1.0 (when templating was introduced)

sisimomo and others added 21 commits October 26, 2021 00:44
…con set 6 Outlined Animated

Marking Dark Sky's animated icon set as legacy only, so options "useAnimatedIcons" and "animateMainIconOnly". You should now use the new animated icons sets.

Adding a new config option "mainIconset". To be used instead of the "animateMainIconOnly" option. Allow you to set icon set for only the current weather icon.

Correcting a small class error in file MMM-OpenWeatherMapForecast.njk.

Changing the iconsets.img image to an animated gif to be able to see new animated icons in the README.md.

Big thanks to basmilius who have made available all theses animated icons for free. To see all the available icons, go see is github page https://github.com/basmilius/weather-icons.
Adding new icons sets: "6fa" - Icon set 6 Filled Animated, "6oa" - Icon set 6 Outlined Animated
… payment details, whereas v2.5 does not. to use v3, add "endpoint": "https://api.openweathermap.org/data/3.0/onecall" to module's config in config.js
changed the units and value of precipitation accumulation from millimeters to inches if units are imperial.
Add ```WEATHER_UPDATED``` notification in an attempt to make this work with MMM-RAIN-MAP module
Undo adding the WEATHER_UPDATED notification
Add keywords
indentation
magic mirror > magicmirror2
magicmirror > magicmirror2; michmich > magicmirrororg
magicmirror > magicmirror2
add code type
add code language
Non-substantive changes per recommendations
Update imperial rain/snow accumulation from mm to in
change default. 2.5 being shut down end of May 24.

maybe no longer expose endpoint 

you can add issues section   to forks now.. please do.
remove mainiconset from defaults
@sdetweil
Copy link

well, this fix is incomplete, I only posted the quick for user approach..

error COULD be not nullOR error could be null and response.statusCode COULD be NOT 200
should separate the tests

if(!error){
  if(response.statusCode == 200){ // good
  }
  else (api error)
}else // some request processing error  

MarcLandis and others added 16 commits September 24, 2024 09:21
Bumps [moment](https://github.com/moment/moment) from 2.29.1 to 2.30.1.
- [Changelog](https://github.com/moment/moment/blob/develop/CHANGELOG.md)
- [Commits](moment/moment@2.29.1...2.30.1)

---
updated-dependencies:
- dependency-name: moment
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
…-2.30.1

Bump moment from 2.29.1 to 2.30.1
Bumps [needle](https://github.com/tomas/needle) from 2.5.2 to 3.3.1.
- [Release notes](https://github.com/tomas/needle/releases)
- [Commits](https://github.com/tomas/needle/commits)

---
updated-dependencies:
- dependency-name: needle
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
It's MIT according to the package.json.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants