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.

    resume()

    Scheduled Pinned Locked Moved Development
    12 Posts 4 Posters 5.7k Views 3 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.
    • cowboysdudeC Offline
      cowboysdude Module Developer @johnnyboy
      last edited by cowboysdude

      @johnnyboy No you haven’t offended me. Not jumping on anyone… I’m just a very direct person and I tend to say what I mean.

      So don’t be offended I’m just simply asking you or anyone else what do you suggest gets done?

      johnnyboyJ 1 Reply Last reply Reply Quote 1
      • johnnyboyJ Offline
        johnnyboy @cowboysdude
        last edited by

        This post is deleted!
        cowboysdudeC 1 Reply Last reply Reply Quote 0
        • cowboysdudeC Offline
          cowboysdude Module Developer @johnnyboy
          last edited by

          @johnnyboy I would rather have suggestions to fix it rather then agree… that should have taken up on the https://github.com/MichMich/MagicMirror site…

          johnnyboyJ 1 Reply Last reply Reply Quote 1
          • johnnyboyJ Offline
            johnnyboy @cowboysdude
            last edited by

            This post is deleted!
            cowboysdudeC 1 Reply Last reply Reply Quote 0
            • cowboysdudeC Offline
              cowboysdude Module Developer @johnnyboy
              last edited by

              @johnnyboy Good, if it helps I’m all for it :)

              johnnyboyJ 1 Reply Last reply Reply Quote 2
              • johnnyboyJ Offline
                johnnyboy @cowboysdude
                last edited by

                This post is deleted!
                1 Reply Last reply Reply Quote 1
                • A Offline
                  Anhalter42
                  last edited by

                  No intention to complain here, so no reason to argue about that.

                  IMO, the two most obvious solutions are

                  1. changing the behaviour, i.e. implement a check whether the module was visible before the call
                  2. changing the description in the documentation so that it matches the behaviour

                  The question is, how do you solve something like this here? Who decides which solution should be preferred and how do I trigger a fix (I dont want to dive into the main core code myself). Do you usually open an issue on github in such cases? Also, the author of this peace of code may have relevant information, but (as usual in open-source projects) its nearly impossible to find out who to contact…

                  brobergB 1 Reply Last reply Reply Quote 0
                  • brobergB Offline
                    broberg Project Sponsor @Anhalter42
                    last edited by

                    @Anhalter42 Yes, open an issue on github or if you already have made changes to either the documentation or written code that does your first suggestion then you would suggest a Pull Request to implement the changes you’ve made

                    1 Reply Last reply Reply Quote 0
                    • 1
                    • 2
                    • 1 / 2
                    • 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