-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Upgrade frontend stack #1125
Comments
I feel it hard to understand frontend codes, so I will be happy when refactor frontend codes. |
Might be useful: http://ngmigrate.telerik.com/from-ng-controller-to-component-classes |
@arikfr I saw you're about to work on this, out of curiosity are you switching to yarn instead of npm? |
@eiso not sure. I might give it a try and if it just works will try to use it, but it's less of a priority right now. |
@arikfr ... Rewrite all components as React components.. at least one so that people who want to contribute and add react component in a plugin manner like grafana in redash can do it. You should reopen this issue as this is not finished. |
It's never too late to use React 👌. |
@oliviertassinari It's already in progress 😉 |
Our frontend stack is becoming more and more outdated which makes keeping up with best practices harder. Once we start actually working on this, we should probably split this into multiple steps.
angular.module
,service
, etc) by using regular Javascript modules.The text was updated successfully, but these errors were encountered: