Update to Angular version 8

Bug #1830973 reported by Bill Erickson on 2019-05-29
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Undecided
Unassigned
Bill Erickson (berick) on 2019-05-29
Changed in evergreen:
status: New → In Progress
Bill Erickson (berick) wrote :

Note to self:

https://update.angular.io/#7.2:8.0

Going to pause this and give project dependencies (e.g. ng-bootstrap) some time to catch up.

Changed in evergreen:
assignee: Bill Erickson (berick) → nobody
status: In Progress → New
Bill Erickson (berick) wrote :

ng-bootstrap now officially supports Angular 8. I have pushed a working branch that migrates current master from Angular 7 to Angular 8. It's a messy branch at the moment and needs a lot more testing, but wanted to get it posted:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/berick/lp1830973-angular-8-wip

Note it requires update Node (see changes in Makefile.install) and a 'rm -rf node_modules' and 'npm install'

Angular dev/prod, Angular tests and ng-lint all pass. Similarly AngularJS building does not appear to be negatively impacted by the NodeJS update.

Will test more and clean up branch soon.

Bill Erickson (berick) wrote :

Beware any Angular branches merged after this branch will require updates to be Angular8-compatible. There's a migration routine built in to automate the process (though in some cases it may require manual intervention).

The command:

ng update @angular/core --from 7 --to 8 --migrate-only

And when we do decide to flip the switch, I can redo the migration on current master.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers