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
chore(core): update ngx-stripe module to angular 9 #35
base: master
Are you sure you want to change the base?
Conversation
Wait, I will try to make a full upgrade toward Angular 9 this morning. |
Done, waiting reviews. I think this needs to be tested with angular 7 and 8 apps. If someone can, please try to run your app with this module |
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.
Not merging at this point as it is unneeded. This module works with angular 7/8 and the ivy deprecation makes this uncessary. I will upgrade when it is necessary.
"tslib": "^1.10.0", | ||
"tslint": "~5.20.0", | ||
"typescript": "~3.5.3", | ||
"typescript": "~3.7.5", |
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.
Angular 9 is not yet compatible with typescript 3.7.5 please use ng update in the future when trying to make changes like this.
@@ -38,26 +38,26 @@ | |||
"doctoc": "doctoc README.md --maxlevel 3" | |||
}, | |||
"peerDependencies": { | |||
"@angular/common": "~8.2.9", | |||
"@angular/common": "~9.0.1", | |||
"@angular/core": "~8.2.9" |
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.
@angular/core should also be bumped
Just updated the moduleWithProviders type. Works fine here.
Updating the application is more tricky. We may have to stop using ng-packagr, because it mess with its typescript version.