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

Uncaught SyntaxError: The requested module does not provide an export named 'transition' #4

Closed
surjithctly opened this issue Nov 30, 2022 · 7 comments · Fixed by #5
Closed

Comments

@surjithctly
Copy link
Contributor

I'm using the direct cdn method.

import { transition } from "https://unpkg.com/vue-petite-transition@2.0.0/dist/vue-petite-transition.es.js";

But when I add it's throwing the following error:

Uncaught SyntaxError: The requested module does not provide an export named 'transition'

See minimal reproduction on codepen: https://codepen.io/surjithctly/pen/qBKMYWr

Can you provide any guidance?

@q1b
Copy link
Owner

q1b commented Dec 1, 2022

Thanks, for reporting it.

I haven't tested It before, let me check.

@q1b
Copy link
Owner

q1b commented Dec 1, 2022

It should work although, but not working must I am doing something wrong.

@q1b
Copy link
Owner

q1b commented Dec 1, 2022

Okay, find the issue.

<script type="module">
  import { createApp } from 'https://unpkg.com/petite-vue?module'
  import { transitionDirective } from "https://unpkg.com/vue-petite-transition@2.0.0/dist/vue-petite-transition.es.js";
  createApp().directive("transition", transitionDirective).mount()
</script>

this is working.

@q1b
Copy link
Owner

q1b commented Dec 1, 2022

It's a quick fix

@q1b
Copy link
Owner

q1b commented Dec 1, 2022

I need to document it, If everything is working for you, you can close the issue.
Or ask any other question.

@surjithctly
Copy link
Contributor Author

yes, it's working, Thank you. The documentation seems wrong.

surjithctly added a commit to surjithctly/petiteVueTransition that referenced this issue Dec 2, 2022
refactoring docs

fixes q1b#4
@surjithctly surjithctly mentioned this issue Dec 2, 2022
@surjithctly
Copy link
Contributor Author

Added a PR that closes this issue. You can merge it if it looks good :)

@q1b q1b closed this as completed in #5 Dec 2, 2022
neterm pushed a commit to neterm/petiteVueTransition that referenced this issue Jan 9, 2023
refactoring docs

fixes q1b#4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants