Git Product home page Git Product logo

Comments (15)

gilest avatar gilest commented on August 24, 2024 1

Yes with staticHelpers: true it seems you must install the ember-tether package even if you don't use tethering.

from ember-modal-dialog.

lukemelia avatar lukemelia commented on August 24, 2024 1

As of 4.0.0, it should be compatible with Ember 4 + Embroider

from ember-modal-dialog.

lukemelia avatar lukemelia commented on August 24, 2024 1

@lukemelia can you achieve this with ember-try and @embroider/macros? Curious because I'm thinking of doing something similar

I don't see any reason that wouldn't work.

from ember-modal-dialog.

ChristopherConnock avatar ChristopherConnock commented on August 24, 2024

when staticComponents is set to true, I get a more specific error:

Build Error (PackagerRunner) in node_modules\ember-modal-dialog\templates\components\modal-dialog.hbs

Module Error (from C:/Users/chris/Documents/GitHub/roast-ember/node_modules/@embroider/webpack/src/webpack-hbs-loader.js):
Unsafe dynamic component modalDialogComponentName in $TMPDIR\embroider\792ec2\node_modules\ember-modal-dialog\templates\components\modal-dialog.hbs

from ember-modal-dialog.

zion03 avatar zion03 commented on August 24, 2024

With staticHelpers: true also

Module Error (from /app/node_modules/@embroider/hbs-loader/src/index.js):
Missing component or helper: ember-tether in node_modules/ember-modal-dialog/templates/components/tether-dialog.hbs

from ember-modal-dialog.

RobbieTheWagner avatar RobbieTheWagner commented on August 24, 2024

I'm seeing Unsafe dynamic component: this.modalDialogComponentName in $TMPDIR/embroider/4a2375/node_modules/ember-modal-dialog/templates/components/modal-dialog.hbs/modal-dialog.hbs

from ember-modal-dialog.

RobbieTheWagner avatar RobbieTheWagner commented on August 24, 2024

@lukemelia are there any plans to make this addon embroider and/or Ember 4 compatible?

from ember-modal-dialog.

lukemelia avatar lukemelia commented on August 24, 2024

@rwwagner90 I definitely intend for it to be compatible with Embroider and Ember 4. I haven't gotten there on my apps yet, so haven't been able to prioritize that work, but I am happy to support anyone who wants to take it on.

from ember-modal-dialog.

lukemelia avatar lukemelia commented on August 24, 2024

This should be fixed in 4.0.0-beta.0, thanks to @rwwagner90

from ember-modal-dialog.

RobbieTheWagner avatar RobbieTheWagner commented on August 24, 2024

The issue with ember-tether may persist. I think we'll have to add it as a peer dep. Will open a PR for that now.

from ember-modal-dialog.

RobbieTheWagner avatar RobbieTheWagner commented on August 24, 2024

#371

from ember-modal-dialog.

RobbieTheWagner avatar RobbieTheWagner commented on August 24, 2024

@lukemelia I think we still need #371 or people will all have to install ember-tether themselves

from ember-modal-dialog.

lukemelia avatar lukemelia commented on August 24, 2024

@rwwagner90 I started looking at #371, and in order to have test coverage working well with optional peer dependencies, I think we'll need to move to the v2 addon monorepo pattern, with a test app that has optional deps installed and one that doesn't. I created #373 to track.

from ember-modal-dialog.

RobbieTheWagner avatar RobbieTheWagner commented on August 24, 2024

@lukemelia perhaps I am misunderstanding how it works, but can't we just ship it as a normal peerDependency? The users will get a warning that they need to install it, but they shouldn't actually have to, I think.

from ember-modal-dialog.

gilest avatar gilest commented on August 24, 2024

test app that has optional deps installed and one that doesn't

@lukemelia can you achieve this with ember-try and @embroider/macros? Curious because I'm thinking of doing something similar

from ember-modal-dialog.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.