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

refactor/vue webpack aliases #360

Closed
wants to merge 1 commit into from
Closed

refactor/vue webpack aliases #360

wants to merge 1 commit into from

Conversation

iaacosta
Copy link
Contributor

@iaacosta iaacosta commented May 28, 2021

Context

Rails uses the es-module version of Vue when installing it with rails webpacker:install, which resolves to the vue/dist/vue.esm.js file.

This brings some strange behaviors when using libraries that import the default Vue instance (just vue).

Changes

Add a Webpack alias inside the configuration to resolve all 'vue' imports to the es-module version.

Closes #359

@iaacosta iaacosta force-pushed the refactor/vue-alias branch from e6f4c23 to 39cab74 Compare May 28, 2021 19:34
@iaacosta iaacosta requested a review from rjherrera May 28, 2021 19:36
Copy link
Member

@rjherrera rjherrera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

aparte de lo que hablamos el otro día se me había quedado un comentario en el tintero jaja

@@ -8,6 +8,7 @@ Features:
Fixes:
- Explicitly install project specified node version in CircleCI build [#352](https://github.com/platanus/potassium/pull/352)
- Update marcel gem (shrine's mime type analyzer) to avoid mimemagic dependency [#357](https://github.com/platanus/potassium/pull/357)
- Add Vue alias in the Webpacker config to avoid strange behaviors of external packages [#360](https://github.com/platanus/potassium/pull/358)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

es el 360 jaja

@iaacosta
Copy link
Contributor Author

This was already implemented on the Typescript + Vue 3 upgrade, so I'm gonna close it

@iaacosta iaacosta closed this Sep 14, 2022
@iaacosta iaacosta deleted the refactor/vue-alias branch September 14, 2022 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Alias 'vue' imports to 'vue/dist/vue.esm.js'
2 participants