MMM-euro2024

[0m4r]

A few weeks more to go, but the magic mirror module is already here for you: https://github.com/0m4r/MMM-euro2024!

[KristjanESPERANTO]

@0m4r Nice! Thanks! 👏
Don’t forget to add it to the module list 🙂

[chrisfr1976]

@0m4r There is a problem with the token. I think, I’m stupid, but:

You write for the config token: [YOUR_TOKEN]

So I’ve tried token:[1234567890], With this I get an error in the config.js and the mirror does not start up.
Then I’ve tried token:1234567890,. Same issue.
Then I’ve tried token:"1234567890", Then the config.js is okay and electron starts. But in this case I get an error message:
[ERROR] MMM-euro2024 fetchTeams 2018 {"message":"Your API token is invalid.","errorCode":400}

…of course I received a real API-key from Daniel. Copy+paste, so the key real in my config is okay.

How should I enter the API-Key? :thinking_face:

[sdetweil]

@chrisfr1976 what is the module property for the api key? that is what ‘token’ means.

and typically [apikey]
means replace all that with the quoted string of the apikey

[0m4r]

@chrisfr1976 can you share the config you set up for the module (obviously hiding the token)? Apart from a problem with the token itself, I cannot think of anything wrong with what you describe.

[sdetweil]

@0m4r the sample is missing the config:{} section

{
  disabled: false,
  module: "MMM-euro2024",
  position: "bottom_center",
  defaults: {
    updateInterval: 60 * 1000, // one minute
  },
  token: [YOUR_TOKEN]
}

should be

{
  disabled: false,
  module: "MMM-euro2024",
  position: "bottom_center",
  config:{
       updateInterval: 60 * 1000, // one minute
       token: [YOUR_TOKEN]
   }
}

your module passes this.config to the node_helper, but there is no config block, so only {} gets passed

this.sendSocketNotification(this.name + 'CONFIG', this.config);

[sdetweil]

@0m4r
also, the defaults go in the modulename.js not in the node_helper.js

defaults: {
    competitionId: 2018, // Euro 2024
    updateInterval: 60 * 1000, // 1 minute
  },

this provides overridable properties via config.js

from the doc
https://docs.magicmirror.builders/development/core-module-file.html#available-module-instance-properties

[0m4r]

@sdetweil thanks for the detailed feedback.
I will update it as soon as I get a moment :)

[0m4r]

The README.md is updated.
The configuration instructions should be now correct: https://github.com/0m4r/MMM-euro2024?tab=readme-ov-file#configuration

[sdetweil]

@0m4r i want to nudge you a little more, on the defaults.

my MMM-Config module presents a form for config customization, and it does this by discovering the parameters in the defaults section of the modulename.js. as it sits, your module would have no parameters for the user to configure

just move that one section to the modulename.js from the node helper and all will be good. no other change to your module is required because you did that change.

[robiv8]

I get a black screen when I activate the module?
Original Config as in description and token available.
Does anyone know what this could be?

Ich bekomme einen Schwarzen Schirm wenn ich das Modul Aktiviere?
Original Config wie in Beschreibung und Token vorhanden.
Weiß jemand was das sein könnte?

Danke
Robert

[0m4r]

@robiv8 hey, I am sorry to hear that.
Are you able to provide some logs?

[0m4r]

@sdetweil this is done in version 1.0.4

[robiv8]

@0m4r
das log sagt: Please correct syntax errors at or above this line: ReferenceError: b1e6…meinToken…67c8c5 is not defined

[0m4r]

@robiv8 I think your configuration is wrong.
Is the token wrapped into quotes?

{
  disabled: false,
  module: 'MMM-euro2024',
  position: 'center',
  header: 'Euro 2024',
  config: {
    updateInterval: 60 * 1000,
    token: "M4g1cM1rr0r"
  }
},

[sdetweil]

@0m4r this is what your module config looks like in MMM-Config

[0m4r]

@sdetweil this is good, isn’t it?

[sdetweil]

@0m4r yes… it works

[robiv8]

@0m4r
Hallo.
This i my config, as described in GitHub

{
	disabled: false,
	module: "MMM-euro2024",
	position: "center",
	header: "Euro 2024",
	config: {
	updateInterval: 60 * 1000,
	token: [b1e67..myToken..67c8c5],
	}
},

Ok Danke,
with superscripts it works.
Why isn’t that right in GitHub?

[0m4r]

@robiv8 try with this:

{
	disabled: false,
	module: "MMM-euro2024",
	position: "center",
	header: "Euro 2024",
	config: {
	updateInterval: 60 * 1000,
	token: "b1e67..myToken..67c8c5",
	}
},