-
Notifications
You must be signed in to change notification settings - Fork 877
docs(ugprade): bring phonecat upgrade tutorial up to speed with modernized Angular 1.x PhoneCat #1106
Conversation
@teropa is this ready to merge? |
@naomiblack Ideally we'd merge it around the same time with angular/angular.js#14416. @petebacondarwin what are your current plans for that? |
It's better not to merge it yet. I'll ping you once angular/angular.js#14416 is merged (or ready to be merged). |
Thanks for the clarification! |
@teropa - how does this relate to the new world version? My inclination is to stop dev here. |
@wardbell This is now out of date. I'll need to rebase it. |
…nized Angular 1.x PhoneCat Relates to angular/angular-phonecat#326
eb9d9bd
to
11b8e2e
Compare
@gkalpak We're now up-to-date with rc.1 and new world packacing here. Ready when you are! 😄 |
@gkalpak @petebacondarwin Would you mind if we land this on angular.io ahead of you? I'm off for vacation after next week and would like to get this and a few other changes to the ugprade guide in before that. |
Just checking with @gkalpak - there may be a couple of small changes that need to be accommodated before we merge the tutorial PR. Will let you know before tomorrow |
@teropa, the main PRs have landed. You are good to go 😃 |
@gkalpak Great, thanks! I'm working through some other changes to upgrade, so I can take care of the additional changes as well. |
Closing this unmerged. The required changes are covered in #1538 |
WIP - Do not merge until Tero says "go"
Relates to angular/angular-phonecat#326
Relates to angular/angular.js#14416