Skip to content
This repository has been archived by the owner on Nov 3, 2021. It is now read-only.

Can't resolve velocity on create-react-app v2 #244

Closed
mustafahlvc opened this issue Sep 27, 2018 · 5 comments
Closed

Can't resolve velocity on create-react-app v2 #244

mustafahlvc opened this issue Sep 27, 2018 · 5 comments

Comments

@mustafahlvc
Copy link

Hi,

I ve tested velocity-react on create-react-app@next ,
create-react-app 2.0.0 will be stable soon:
facebook/create-react-app#5103 (comment)

I m getting Can't resolve 'velocity' error.

I m not sure but it could be related with webpack and updating velocity-animate version could solve this issue? julianshapiro/velocity#743 (comment)

Could you please checkout the issue?

@fionawhim
Copy link
Collaborator

Where are you seeing that? What version?

The latest version of velocity-react has require('velocity-animate'). See velocity-animate-shim.js

@mustafahlvc
Copy link
Author

I'm getting the error using "react-scripts": "2.0.0" and importing as below:

import {VelocityComponent} from 'velocity-react';
import 'velocity-animate';
import 'velocity-animate/velocity.ui';

Thanks in advance.

@fionawhim
Copy link
Collaborator

What version of velocity-react? Is there a stack trace for the resolve error?

@mustafahlvc
Copy link
Author

"velocity-react": "1.4.1"

image

When i add "velocity-animate": "^2.0.5" manually it works but face off the ui pack issue due to multiple velocity-animate package.

If, even with the above statements, you see errors like Velocity: First argument (transition.shrinkIn) was not a property map, a known action, or a registered redirect. Aborting. it is likely that there are 2 copies of velocity-animate in your node_modules. Use npm dedupe to collapse them down to one.

@mustafahlvc
Copy link
Author

Hi,

The issue solved with the release "react-scripts": "2.0.4".

Thanks.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants