Skip to content
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

resolveDeepImport does not support aliased packages #4957

Closed
7 tasks done
KaelWD opened this issue Sep 17, 2021 · 1 comment · Fixed by #4961
Closed
7 tasks done

resolveDeepImport does not support aliased packages #4957

KaelWD opened this issue Sep 17, 2021 · 1 comment · Fixed by #4961
Labels
p3-downstream-blocker Blocking the downstream ecosystem to work properly (priority)

Comments

@KaelWD
Copy link
Contributor

KaelWD commented Sep 17, 2021

Describe the bug

import 'vuetify/lib/components/VApp/index.mjs' with vuetify being "vuetify": "npm:@vuetify/nightly@next" fails with

[vite] Internal server error: Missing ".onents/VApp/index.mjs" export in "@vuetify/nightly" package

This line:

id = '.' + id.slice(data.name.length)

is using the original package name instead of the alias, resulting in mangled paths.

id: 'vuetify/lib/components/VApp/index.mjs'
data.name: '@vuetify/nightly'

Reproduction

N/A

System Info

N/A

Used Package Manager

yarn

Logs

No response

Validations

@Shinigami92 Shinigami92 added bug contribution welcome p3-downstream-blocker Blocking the downstream ecosystem to work properly (priority) and removed pending triage labels Sep 17, 2021
@github-actions
Copy link

Hello @KaelWD. We like your proposal/feedback and would appreciate a contribution via a Pull Request by you or another community member. We thank you in advance for your contribution and are looking forward to reviewing it!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
p3-downstream-blocker Blocking the downstream ecosystem to work properly (priority)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants