Sitelet https://web.archive.org/web/20210111231506/https://github.com/akveo/ngx-admin/pull/5727
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

Update to angular v10 #5727

Open
wants to merge 7 commits into
base: master
from
Open

Update to angular v10 #5727

wants to merge 7 commits into from

Conversation

@nhaancs
Copy link

@nhaancs nhaancs commented Jul 14, 2020

Please read and mark the following check list before creating a pull request (check one with "x"):

Short description of what this resolves:

"intl": "1.2.5",
"ionicons": "2.0.1",
"leaflet": "1.2.0",
"nebular-icons": "1.1.0",
"ng2-ckeditor": "^1.2.2",
"ng2-ckeditor": "^1.2.7",

This comment has been minimized.

@jainAdijain

jainAdijain Jul 17, 2020

Should we update the this package when migrating it to next stable version ?
Curious to know if it wont affect the working ?

This comment has been minimized.

@nhaancs

nhaancs Jul 20, 2020
Author

The current version has incompatible dependency so I upgraded it

@jainAdijain
Copy link

@jainAdijain jainAdijain commented Jul 17, 2020

Hey @nhaancs ,

Cursious, to know did you followed any process to migrate it to next stable version of Angular?
If you can share any pointers ?

Thanks,
Aditya

@nhaancs
Copy link
Author

@nhaancs nhaancs commented Jul 20, 2020

Hey @nhaancs ,

Cursious, to know did you followed any process to migrate it to next stable version of Angular?
If you can share any pointers ?

Thanks,
Aditya

I followed the guidelines from https://update.angular.io. Some packages have incompatible dependencies and I updated them too.

Copy link

@jainAdijain jainAdijain left a comment

Approved!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.