-
Notifications
You must be signed in to change notification settings - Fork 162
Support angular 4 #404
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
Support angular 4 #404
Conversation
package.json
Outdated
| "devDependencies": { | ||
| "devextreme": "~16.2.4", | ||
| "jquery": "^2.0.0 || ^3.0.0", | ||
| "jquery": "^2.0.0 || ~3.1.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this unnecessary dev dependency
package.json
Outdated
| "@types/jquery": "^2.0.0 || ^3.0.0", | ||
| "jquery": "^2.0.0 || ^3.0.0" | ||
| "@types/jquery": "^2.0.0 || ~3.1.1", | ||
| "jquery": "^2.0.0 || ~3.1.1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems that "2.0.0 - 3.1" is more suitable version specification
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we need these changes?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@GoshaFighten These changes fixed issue #406
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, but we will be stuck to jQuery 3.1.1. jQuery is going to fix the issue in $.holdReady is broken in 3.2.0, plus we are going fix it in our code in the DevExtreme fails with the "$.holdReady is not a function" error when jQuery 3.2.0 is used bug report.
No description provided.