MagicMirror Forum
    • Recent
    • Tags
    • Unsolved
    • Solved
    • MagicMirror² Repository
    • Documentation
    • 3rd-Party-Modules
    • Donate
    • Discord
    • Register
    • Login
    A New Chapter for MagicMirror: The Community Takes the Lead
    Read the statement by Michael Teeuw here.

    MMM-SoccerLiveScore

    Scheduled Pinned Locked Moved Sport
    187 Posts 37 Posters 288.9k Views 35 Watching
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • karsten13K Offline
      karsten13 @np27np27
      last edited by

      @np27np27

      https://forum.magicmirror.builders/topic/15778/fix-for-black-screen-in-2-16-and-later

      0m4r0 1 Reply Last reply Reply Quote 0
      • 0m4r0 Offline
        0m4r Module Developer @np27np27
        last edited by

        @np27np27 the API the module is fetching the data from does not seem to respond anymore (or at least, they do not in most of the cases).
        I have tried with Postman to double-check if it is a problem with my code or of the API, but also with Postman I get the same error/problem.

        I am trying to figure if I can replace the API with others, but I have not found anything I can really move without losing some of the information or the functionalities implemented now.

        1 Reply Last reply Reply Quote 0
        • 0m4r0 Offline
          0m4r Module Developer @karsten13
          last edited by

          @karsten13 this could be one reason (I plan to switch to node-fetch), but I experience the same problem as reported by @np27np27 and I have detected the APIs are responding with errors.

          Unfortunately, there is not much I can do about it :(

          N 1 Reply Last reply Reply Quote 1
          • 0m4r0 Offline
            0m4r Module Developer @np27np27
            last edited by

            @np27np27 can you try to check out from master now and let me know if the problem is gone? :D

            1 Reply Last reply Reply Quote 0
            • N Offline
              np27np27 @0m4r
              last edited by

              @0m4r
              Yes it works now! Thanks for the quick help and update ;)

              1 Reply Last reply Reply Quote 1
              • N Offline
                np27np27
                last edited by

                @0m4r
                Actually I found another issue after the fix. The scores don’t seem to be updating during games or even at regular intervals.
                I’m not sure if it’s updating at all, maybe only when magicmirror restarts or if I restart the module. @0m4r can you have a look and confirm if it works for you?

                0m4r0 1 Reply Last reply Reply Quote 0
                • 0m4r0 Offline
                  0m4r Module Developer @np27np27
                  last edited by

                  @np27np27 I guess you are correct, something is wrong since the second last update.
                  I have tried to reduce the number of API calls trying to have them issued only when games are on or about to start rather than each and every 2 mins… but sometimes things do not work as I expect.

                  You should be able to see a message at the bottom of the standings telling you when the next API request is expected to be executed, that should help d figure if the new data are requested at a meaningful time.

                  66bb1a3e-af5e-4b1c-8ec6-32d89771e970-image.png

                  N 1 Reply Last reply Reply Quote 0
                  • N Offline
                    np27np27 @0m4r
                    last edited by

                    @0m4r
                    For me the module has an api request date of yesterday! So it’s probably not called the api?

                    1 Reply Last reply Reply Quote 0
                    • 0m4r0 Offline
                      0m4r Module Developer
                      last edited by

                      @np27np27 probably. I have realized I have a mistake when the event is finished and the next date should be set to be used…

                      If you manually refresh with F5 or restart does it get any better?

                      N 1 Reply Last reply Reply Quote 0
                      • N Offline
                        np27np27 @0m4r
                        last edited by

                        @0m4r
                        Yes if you manually restart MM the the data updates.

                        1 Reply Last reply Reply Quote 0
                        • 0m4r0 Offline
                          0m4r Module Developer
                          last edited by

                          @np27np27 alright - I will try to fix it, but it may take some time, I usually dedicate time to my side project on the weekend

                          N 1 Reply Last reply Reply Quote 0
                          • N Offline
                            np27np27 @0m4r
                            last edited by

                            @0m4r
                            No problem, thanks for trying and keeping this module going 👍🏾👍🏾

                            0m4r0 1 Reply Last reply Reply Quote 1
                            • 0m4r0 Offline
                              0m4r Module Developer @np27np27
                              last edited by 0m4r

                              @np27np27 can you try check out and run this branch:
                              https://github.com/0m4r/MMM-SoccerLiveScore/tree/feature/node-fetch

                              > cd [YOUR_PATH_TO_MMM-SoccerLiveScore]
                              > git fetch -p
                              > git checkout feature/node-fetch 
                              > git pull -r
                              > npm ci
                              > [RESTART MM]
                              

                              I did do some major changes (removed request for node-fetch, and some other code improvements amongst which I hope to have fixed the problem of the refresh you reported)

                              N 1 Reply Last reply Reply Quote 0
                              • N Offline
                                np27np27 @0m4r
                                last edited by np27np27

                                @0m4r said in MMM-SoccerLiveScore:

                                @np27np27 can you try check out and run this branch:
                                https://github.com/0m4r/MMM-SoccerLiveScore/tree/feature/node-fetch

                                > cd [YOUR_PATH_TO_MMM-SoccerLiveScore]
                                > git fetch -p
                                > git checkout node-fetch
                                > git pull -r
                                > npm ci
                                > [RESTART MM]
                                

                                I did do some major changes (removed request for node-fetch, and some other code improvements amongst which I hope to have fixed the problem of the refresh you reported)

                                Hi @0m4r ,

                                Ive tried the above but got an error at the checkout node-fetch line. Can’t find the file or something similar?
                                Still continued on and did the rest.
                                The mm restart has updated the module data. I will check when a game plays tonight to see if the live data changes.

                                0m4r0 1 Reply Last reply Reply Quote 0
                                • 0m4r0 Offline
                                  0m4r Module Developer
                                  last edited by

                                  This post is deleted!
                                  1 Reply Last reply Reply Quote 0
                                  • 0m4r0 Offline
                                    0m4r Module Developer @np27np27
                                    last edited by

                                    @np27np27 said in MMM-SoccerLiveScore:

                                    git checkout node-fetch

                                    should be:

                                    git checkout feature/node-fetch

                                    1 Reply Last reply Reply Quote 0
                                    • N Offline
                                      np27np27
                                      last edited by np27np27

                                      @0m4r
                                      OK tried again, this time I received the following after npm ci

                                      npm WARN prepare removing existing node_modules/ before installation
                                      npm ERR! cipm can only install packages when your package.json and package-lock.json or npm-shrinkwrap.json are in sync. Please update your lock file with npm install before continuing.
                                      npm ERR!
                                      npm ERR!
                                      npm ERR! Invalid: lock file’s node-fetch@2.6.6 does not satisfy node-fetch@v2.6.1
                                      npm ERR!

                                      npm ERR! A complete log of this run can be found in:
                                      npm ERR! /home/pi/.npm/_logs/2021-12-06T22_20_06_917Z-debug
                                      .log

                                      N 1 Reply Last reply Reply Quote 0
                                      • N Offline
                                        np27np27 @np27np27
                                        last edited by

                                        @np27np27 said in MMM-SoccerLiveScore:

                                        @0m4r
                                        OK tried again, this time I received the following after npm ci

                                        npm WARN prepare removing existing node_modules/ before installation
                                        npm ERR! cipm can only install packages when your package.json and package-lock.json or npm-shrinkwrap.json are in sync. Please update your lock file with npm install before continuing.
                                        npm ERR!
                                        npm ERR!
                                        npm ERR! Invalid: lock file’s node-fetch@2.6.6 does not satisfy node-fetch@v2.6.1
                                        npm ERR!

                                        npm ERR! A complete log of this run can be found in:
                                        npm ERR! /home/pi/.npm/_logs/2021-12-06T22_20_06_917Z-debug
                                        .log

                                        I did npm install then npm Ci and it seems no errors this time. Let’s see if it works now. I’ll let you know 👍🏾

                                        N 1 Reply Last reply Reply Quote 0
                                        • N Offline
                                          np27np27 @np27np27
                                          last edited by

                                          @0m4r,

                                          Looks like it’s still not working. It’s gone past the api call date that is displayed at the bottom of the module and it’s not updated. Does it work for you?

                                          0m4r0 1 Reply Last reply Reply Quote 0
                                          • 0m4r0 Offline
                                            0m4r Module Developer @np27np27
                                            last edited by

                                            @np27np27 can you try pulling the code once more? I did mess up the logic to re-issue the API requests, but I hopefully fixed it now!

                                            N 1 Reply Last reply Reply Quote 0

                                            Hello! It looks like you're interested in this conversation, but you don't have an account yet.

                                            Getting fed up of having to scroll through the same posts each visit? When you register for an account, you'll always come back to exactly where you were before, and choose to be notified of new replies (either via email, or push notification). You'll also be able to save bookmarks and upvote posts to show your appreciation to other community members.

                                            With your input, this post could be even better 💗

                                            Register Login
                                            • 1
                                            • 2
                                            • 3
                                            • 4
                                            • 5
                                            • 6
                                            • 7
                                            • 8
                                            • 9
                                            • 10
                                            • 5 / 10
                                            • First post
                                              Last post
                                            Enjoying MagicMirror? Please consider a donation!
                                            MagicMirror created by Michael Teeuw.
                                            Forum managed by Sam, technical setup by Karsten.
                                            This forum is using NodeBB as its core | Contributors
                                            Contact | Privacy Policy