-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Stack monitoring] Angular migration clean-up #109791
Comments
Pinging @elastic/logs-metrics-ui (Team:logs-metrics-ui) |
I moved the flag removal to #115504 |
Also moved the setup mode refactor/test points to #112512 |
@matschaffer I moved the refactor/test points back here. What is mentioned in this issue regarding the setup mode should be cleaned now unless there is a major reason to not do it. It should be quick and easy. (I was planning to finish the remaining bullet points between today and tomorrow 😅 ) The refactor setup mode task that was in the tech debt issue is to rewrite how the setup mode is implemented, because, in my opinion, the way it's currently implemented is complex and confusing probably because of the mix of Angular and React. |
@estermv - cool. I'll leave it in your hands then. Seems like we might be done as far as upstream is concerned but you probably know better than I do. |
Closing this as all tasks have been completed. |
Required to clean to remove angular app from Stack Monitoring
Required to remove Angular from
package.json
getAngularInjector
#115499After angular is removed
The text was updated successfully, but these errors were encountered: