Skip to content
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

[Monitoring] Replace all the Angular directives with React components #19052

Closed
tsullivan opened this issue May 14, 2018 · 1 comment
Closed
Assignees

Comments

@tsullivan
Copy link
Member

These <monitoring-cluster-xyz-status> Angular directives can be rewritten to use React components instead of Angular HTML. That way when the view gets rewritten to use React, the status bar component will already be in the code and ready to go in.

> public/views/kibana/overview/index.html:6:23:  <monitoring-cluster-status-kibana status="pageData.clusterStatus"></monitoring-cluster-status-kibana>
  public/views/kibana/instances/index.html:6:23:  <monitoring-cluster-status-kibana status="kibanas.data.clusterStatus"></monitoring-cluster-status-kibana>
  public/views/logstash/overview/index.html:2:23:  <monitoring-cluster-status-logstash status="pageData.clusterStatus"></monitoring-cluster-status-logstash>
  public/views/logstash/nodes/index.html:2:23:  <monitoring-cluster-status-logstash status="lsNodes.data.clusterStatus"></monitoring-cluster-status-logstash>
  public/views/logstash/pipelines/index.html:6:23:  <monitoring-cluster-status-logstash status="pageData.clusterStatus"></monitoring-cluster-status-logstash>
  public/views/elasticsearch/ml_jobs/index.html:2:23:  <monitoring-cluster-status-elasticsearch status="mlJobs.data.clusterStatus"></monitoring-cluster-status-elasticsearch>

@tsullivan tsullivan self-assigned this May 14, 2018
@tsullivan tsullivan changed the title [Monitoring] Replace all the status bar directives with React components [Monitoring] Replace all the Angular directives with React components May 17, 2018
@tsullivan
Copy link
Member Author

tsullivan commented Jun 1, 2018

Broken into multiple PRs:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant