Windy Community
    • Unread
    • Categories
    • Groups
    • Go to windy.com
    • Register
    • Login

    [CLOSED] v39.0.0

    Public beta
    7
    26
    11.5k
    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.
    • rittelsR
      rittels Code contributor | Premium
      last edited by

      @vicb @marekd

      New compiler works well. Thanks Marek.

      Small issue on windows system: https://github.com/windycom/windy-plugins/issues/30

      The autoloading and querystring passing within the client not working yet. I am working on it.

      1 Reply Last reply Reply Quote 1
      • idefix37I idefix37 referenced this topic on
      • vicbV
        vicb Paraglider @marekd | Premium
        last edited by

        @marekd said in v39.0.0:

        We removed W.define and W.require. It will work (for backward compatibility for our own plugins), but be sure you use only import { anything } from '@windy/someModule for the future compatibility

        Ahah you got me here!

        I naively thought that I should have import { anything } from '@windy/someModule; in my code.

        But no the code is still transformed at build time.

        I guess I will need some more time port my plugin.

        When is v39 supposed to be released?

        marekdM 1 Reply Last reply Reply Quote 0
        • marekdM
          marekd Administrator @vicb
          last edited by

          @vicb We found some issues, so I think we will bugfix them this week. So we would like to release the new version next week. Feel free to ask for help if there is anything I can help with.

          vicbV 1 Reply Last reply Reply Quote 0
          • vicbV
            vicb Paraglider @marekd | Premium
            last edited by

            @marekd Are you thinking of releasing the compiler (rollup plugins) as an npm package ? It would be easier for me to first add it to the project and then get updates.

            vicbV 1 Reply Last reply Reply Quote 0
            • vicbV
              vicb Paraglider @vicb | Premium
              last edited by

              @vicb

              Getting there

              fdbe3ee7-fdb8-4302-bce1-5ad64dbf5c71-image.png

              I have some feedback about the new compiler:

              Why do you have rollup add the call to W.loadPlugin(...) instead of having the plugin dev call it?

              Kind of the same questions for both the html and the css. They shouldn't need any special treatment now that you are using rollup. You should be able to remove both buildPluginHtml and buildPluginCss.

              Also why would the build system define the meta object?

              You can have a look at what I did on the published version of my plugin to use rollup (+existing plugins) without having to build a custom solution.

              The only thing that seems useful to me in the new compiler is the rewrite of the imports. But I am not sure I like it either. I preferred to use W.require(...) before than using import ... today which is not a real import.

              I'll work on testing/polishing my plugin for v39 now and I hope I can be ready in time.

              marekdM 1 Reply Last reply Reply Quote 1
              • marekdM
                marekd Administrator @vicb
                last edited by

                @vicb Thanks for the feedback! The only reason was to keep it as compatible as possible with the old version, that is all. We are making things too complicated for the developers already, so we have tried to make it so they do not have to deal with too much around the build.

                Feel free to make this package and take care of it, I would be more than happy if someone takes over and I do not have to worry about it.

                We will remove W global object and W.require as well in a favor of ESM. So do not use it. That is why the new compiler was created, so that we could prepare it for people beforehand and the codes would work properly with minimal changes.

                vicbV 1 Reply Last reply Reply Quote 1
                • marekdM
                  marekd Administrator
                  last edited by

                  BTW, I have merged v39 into master: https://github.com/windycom/windy-plugins

                  1 Reply Last reply Reply Quote 0
                  • rittelsR
                    rittels Code contributor | Premium
                    last edited by marekd

                    @vicb @marekd

                    Beta has been update to include a fix for autoloading the plugins.

                    Now:
                    windy.com/plugins/windy-plugin-abc works again.

                    You can send a querystring like this:
                    windy.com/plugins/windy-plugin-abc?something=123

                    You can then access it within the plugin with onopen function:

                    export const onopen = params => {
                        console.log(params.query)
                    }
                    

                    You can also send params to another plugin that you may wish to open like this:

                    broadcast.emit('rqstOpen', pluginName, {query:{ something: 123}} )
                    

                    Finally: params can also be sent to the dev plugin:
                    www.windy.com/dev?something=1234&query2=1234.

                    1 Reply Last reply Reply Quote 2
                    • vicbV
                      vicb Paraglider @marekd | Premium
                      last edited by

                      @marekd said in v39.0.0:

                      Feel free to make this package and take care of it, I would be more than happy if someone takes over and I do not have to worry about it.

                      I'll take a stab when I understand what is coming with "We will remove W global object and W.require as well in a favor of ESM. So do not use it" and when

                      1 Reply Last reply Reply Quote 1
                      • marekdM marekd locked this topic on
                      • marekdM
                        marekd Administrator
                        last edited by

                        Beta test is closed, v39.0.0 has been released. Thank you all for your help!

                        1 Reply Last reply Reply Quote 3
                        • First post
                          Last post
                        Windyty, S.E. - all rights reserved. Powered by excellent NodeBB
                        NodeBB & contributors, OSM & contributors, HERE maps
                        Terms of Use     Privacy Policy