Does this work with Embroider? about ember-modal-dialog HOT 15 CLOSED

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

from ember-modal-dialog.

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

from ember-modal-dialog.

@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.

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.

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.

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.

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

from ember-modal-dialog.

@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.

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

from ember-modal-dialog.

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.

#371

from ember-modal-dialog.

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

from ember-modal-dialog.

@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.

@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.

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.