search

USACE / groundwork

React Components for USACE Applications
https://usace.github.io/groundwork
4 stars 0 forks source link

remove plotly from groundwork #70 and documentation #75

Closed krowvin closed 2 weeks ago

krowvin commented 2 weeks ago

Solves #70

Did npm uninstall plotly.js-dist and did a Ctrl + Shift + F to ensure no other plot keywords remained in the source files.

Removed all references to plots. For now.

willbreitkreutz commented 2 weeks ago

@krowvin @adamscarberry do we know if anyone is using this yet? Don't want to pull the lib out without having a replacement. There are two options, groundwork-plotly or just docs saying here's how you integrate plotly, it's not really that hard and I think that might meet the need... thoughts?

adamscarberry commented 2 weeks ago

@willbreitkreutz I think it's early enough that we should be ok yanking this with little to no impacts.