-
Notifications
You must be signed in to change notification settings - Fork 357
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
[WIP] Angular 2 Migration Preparation(Draft) #148
Comments
@martinpovolny here is the starter Angular 2 migration issue; feel free to edit... |
From @himdel on December 13, 2016 12:13 @mtho11 great list.. just adding some random thoughts.. :) For point one (yarn), we already have ManageIQ/manageiq#12581 And, maybe you mean it that way, but.. I'd rather do the webpack transition in this release, separate from any Angular 2 efforts. So that we don't move to webpack & angular 2 at the same time :). And agreed we definitely need to look into webpacker since it will be the default in rails 5.1. Another thing, ngUpgrade should probably never get into a production build. Meaning we should ideally start right after Probably agreed on the UI router, although since we're still not using any frontend routing, if at that point we still don't, it may make sense to go with the new default router.. Especially since 1.6 should have a router closer to ng2.. And i18n may be a toughie.. but some gettext version will exist :). |
@himdel sounds like a smart thing to do webpack in this release so like you said to "isolate the Angular 2 efforts". |
From @himdel on December 19, 2016 21:37
Created ManageIQ/manageiq#13255 to address the rx update (but depends on npm) |
From @dclarizio on January 12, 2017 22:34 @mtho11 can you please close this and reopen on the ui-classic repo? Thx, Dan |
FYI: this issue was migrated from |
@AparnaKarve, @h-kataria, @hstastna : can you, please, read/comment this? We need to make sure that we don't introduce new obstacles for Angular 2 in our own PRs as well as in review. From my point of view the Angular 1.x Best Practices are critical for us so that with each PR we don't increase the amount of work that will have to be done to get to Angular 2. Thx! |
I've read this list and I like it. If there will be something to add to the list from my point of view, I will do it. |
Angular 2 integration with Rails as a standard option is now feasible: https://github.com/rails/webpacker#ready-for-angular-with-typescript |
@himdel, @skateman, @AparnaKarve : ^^^ |
Nice, so I guess we'll have to go rails 5.1 first, but hopefully that's the plan anyway. |
This issue has been automatically marked as stale because it has not been updated for at least 6 months. If you can still reproduce this issue on the current release or on Thank you for all your contributions! |
React was chosen over Angular 2/4. @miq-bot close_issue |
From @mtho11 on December 13, 2016 4:17
Purpose
The purpose of this issue is to identify the tasks need for Angular 2 migration. So we can identify task dependencies and blockers and also split up the work on these individual tasks.
At this time, this is a living document so feel free to add items or elaborate or question tasks.
This is an open task for anyone to contribute, so we can just start to get our hands around all the migration issues and create a plan.
Tasks
UI Router 1.0.0+ to bridge to New Angular 2 routerRecommendation
Use Rails 5.1+ for both webpack and Yarn (via webpacker) to make their way into Rails as a standard. This should not stop prototypes(using the above solutions), just that we don’t want to create a non-standard solution for the future.
Starting with ngUpgrade for an interim period should provide a less painful path forward as both Angular 1 and Angular 2 can coexist (albeit at a cost of both runtimes); until everything can be fully converted to Angular 2.
Angular 1.x Best Practices (TBD)
This is probably best off in its own issue as it could be lengthy and have many of its own conversations. (This may not be needed given that we need to migrate to ng2 and the ng1 stuff will exist for a short time period) I cannot compete wtih what is already out there:
Next
Surely, there are things missing here, as the migration is large and complex, however, this should serve as a starting place. Please edit as needed so we can divide these up and conquer.
Copied from original issue: ManageIQ/manageiq#13131
The text was updated successfully, but these errors were encountered: