MMM-Strava

[ianperrin]

@lavolp3 I’ve updated the comment above with clearer notes for beta testers.

Everything you need should be done from the Magic Mirror, so if you can follow these steps again and let me know where the error occurs or your experience differs that would be great.

[lavolp3]

@ianperrin Did it again as described, but still a “Bad Request” Message. Maybe I’m doing something wrong: Error message says invalid redirect_uri, so I guess I need to play around with it a bit more.

[lavolp3]

@ianperrin OK now it works.
I set my ddns domain “xxxx.ddns.net” as callback domain in the Authentication profile.
Then it works. The link for the authorisation inquires that the request is being sent from inside that domain. I had set localhost as the callback domain and made the request from the ddns which in that specific case did not work.

Maybe you could include a description for anyone trying to do the authentication from a browser not on the RasPi itself. You would need to set the callback domain appropriately.

[ianperrin]

@lavolp3 that makes sense, I understand your issue now. Good pick up and we’ll done for finding that solution. I’ll look to update the read me with notes for creating the application as well as authorising the modules.

In the meantime, let me know how the token refresh works, or if any other errors appear.

[lavolp3]

@ianperrin Table mode doesn’t seem to rotate the actual data into monthly data for me. I have done a run of 6.4 km on Tuesday. But much more in Jan and Feb :-)

Can you confirm that?

[ianperrin]

@lavolp3 said in MMM-Strava:

@ianperrin Table mode doesn’t seem to rotate the actual data into monthly data for me. I have done a run of 6.4 km on Tuesday. But much more in Jan and Feb :-)
Can you confirm that?

Hi @lavolp3 - can you post your config for the module and if possible the output from the log?

[lavolp3]

@ianperrin said in MMM-Strava:

rom the log?

My config:

        //disabled: true,
        module: 'MMM-Strava',
        position: 'bottom_right',
        config: {
            client_id: "2xxxx",
            client_secret: "9bdxxxxxxxxxxxxxxxxxxxxxxxxxxxxx",
            // strava_id: '23128899',
            // access_token: 'af1a3e4xxxxxxxxxxxxxxxxxxxxxxxxxxxx',
            // period: 'recent',
            auto_rotate: true,
            animationSpeed: 0,
            mode: "chart",
            debug: true
        }

Regarding logs: This comes from the developer’s tab:

Load script: modules/MMM-Strava//MMM-Strava.js loader.js:182:4
Check MagicMirror version for module 'MMM-Strava' - Minimum version:  2.2.0 - Current version: 2.6.0 module.js:469:3
Module registered: MMM-Strava module.js:477:2
Bootstrapping module: MMM-Strava loader.js:152:3
Scripts loaded for: MMM-Strava loader.js:157:4
Load stylesheet: modules/MMM-Strava/MMM-Strava.css loader.js:197:4
Styles loaded for: MMM-Strava loader.js:159:5
MMM-Strava - Load translation: translations/de.json translator.js:172:5
MMM-Strava - Load translation fallback: translations/en.json translator.js:174:5
Translations loaded for: MMM-Strava loader.js:161:6
Starting module: MMM-Strava MMM-Strava.js:114:9
MMM-Strava received a system notification: ALL_MODULES_STARTED module.js:156:4
MMM-Strava received a system notification: MODULE_DOM_CREATED module.js:156:4
MMM-Strava received a system notification: DOM_OBJECTS_CREATED module.js:156:4
MMM-Strava: "Receiving notification: DATA for module_25_MMM-Strava" MMM-Strava.js:201:13
MMM-Strava received a module notification: DARK_SKY_FORECAST_WEATHER_UPDATE from sender: MMM-DarkSkyForecast

npm start dev does only give me:

MMM-Strava: "Received notification: SET_CONFIG"

[ianperrin]

@lavolp3 - thanks and good spot. Currently rotation is only supported for table mode. I’ve pushed a fix to the develop branch to enforce this behaviour. Please use git pull to get and test the changes.

[lavolp3]

@ianperrin Confirmed! Chart is not rotating anymore.
However it might be worth to think about several rotating charts. Like for weekdays and a range of several months?

[ianperrin]

@lavolp3 said in MMM-Strava:

@ianperrin … it might be worth to think about several rotating charts. Like for weekdays and a range of several months?

Good idea, could you add it as an issue in the Github repository so I don’t forget it?