-
Notifications
You must be signed in to change notification settings - Fork 109
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
allowAppImports
entries are now allowed to import from other allowAppImports
entries.
#615
Comments
does it work using the app name? i.e. |
It does not:
Something interesting I see from this output is:
which seems like a goofy? idk, or maybe I don't know what it's trying to tell me? |
You're missing this config on your repo in the
|
Thanks for pointing that out! after adding that however, I get: ERROR in ./app/util/idk.js 2:0-39
Module not found: Error: Can't resolve '../components/foo.js' in '/home/nvp/Development/tmp/appImports-import-appImports/app/util'
resolve '../components/foo.js' in '/home/nvp/Development/tmp/appImports-import-appImports/app/util'
using description file: /home/nvp/Development/tmp/appImports-import-appImports/package.json (relative path: ./app/util)
Field 'browser' doesn't contain a valid alias configuration with both absolute and relative import paths, with and without the js extension. |
Repro: https://github.com/NullVoxPopuli/ember-auto-import-repro-appImports-import-other-appImports
Scenario:
hi
Config:
Try to import
import './util/idk';
from anywhereNullVoxPopuli/ember-auto-import-repro-appImports-import-other-appImports@3a179e8#diff-322f0482bd28fc40f84fcb0aaf208fef7fb63082d462cf02d11f46e59c531c24R6
NullVoxPopuli/ember-auto-import-repro-appImports-import-other-appImports@9c9bc4f#diff-be9bc2296e644e9cc4c783e51a6a231716c251353aca253effd68264bae589fbR6
Error:
The text was updated successfully, but these errors were encountered: