Make the status graphs look nice in demo mode.

I do not remember why we did not add a dependency to TMGMT core for the chart tools? Any idea?

CommentFileSizeAuthor
chart_tools_in_demo_module_1.patch311 bytescgalli

Comments

miro_dietiker’s picture

miro_dietiker
Location Switzerland
commented

Agree to add the dependency within the demo module.

However, TMGMT core itself just recommends the google chart tools module. We don't require it because we try to be as lightweight as possible. We already have heavy dependencies.

cgalli’s picture

cgalli commented

Ok. So we leave it as is.

Please RBTC.

miro_dietiker’s picture

miro_dietiker
Location Switzerland
commented
Status: Needs review » Reviewed & tested by the community

Works cleanly with simplytest.me

cgalli’s picture

cgalli commented
Status: Reviewed & tested by the community » Fixed

Commited and Pushed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.