Replace charts with react components

[kamils-iRonin]

This closes #20

[theforeman-bot]

Do not merge! This patch has not been tested yet.

Can an existing organization member please verify this patch?

[ohadlevy]

can this somehow be added to the plugin template at https://github.com/theforeman/foreman_plugin_template ?

On Tue, Dec 11, 2018 at 2:17 PM Avi Sharvit notifications@github.com wrote:

@sharvit commented on this pull request.

In .eslintrc https://github.com/theforeman/foreman_omaha/pull/32#discussion_r240586339 :

@@ -0,0 +1,49 @@ +{

• "root": true,

I would strongly recommend using the same configurations as in the core using eslint-plugin-patternfly-react https://github.com/patternfly/patternfly-react/tree/master/packages/eslint-plugin-patternfly-react .

See: theforeman/foreman@839bcbb https://github.com/theforeman/foreman/commit/839bcbbd1499e6dd377ef8cf7f4ba67742c55eed

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/theforeman/foreman_omaha/pull/32#discussion_r240586339, or mute the thread https://github.com/notifications/unsubscribe-auth/AABOx40swVYqNiuyuLp_j9jQFXdGgapYks5u36JjgaJpZM4W-K1v .

[timogoebel]

As discussed with @ekohl on IRC, I'd wait for the packaging issues to be solved before merging this PR.

[timogoebel]

[test foreman_omaha]

[ezr-ondrej]

I believe now it should be much easier to finish this.

[kamils-iRonin]

closed in favor of https://github.com/theforeman/foreman_omaha/pull/55