>From my point of view, you cannot create an apache release w/ this lib in here. This was missed in the 1.0.0 release but that should not have been releasable.
On Tue Dec 16 2014 at 10:50:57 AM Shawn Feldman <[email protected]> wrote: > i think we need to remove the lib, then remove the chart on the application > home page, then verify the tests pass > > On Tue Dec 16 2014 at 8:48:14 AM Dave Johnson <[email protected]> wrote: > > > We should definitely remove it and as soon as we can. I’d like to get an > > RC out for 1.0.1 today if possible. > > > > What needs to be done to remove it? > > > > - Dave > > > > > > > > > > > On Dec 16, 2014, at 10:30 AM, Shawn Feldman <[email protected]> > > wrote: > > > > > > We should probably remove it. It was used in our proprietary codebase > > and I > > > think only serves one chart on the application page > > > > > > On Tue, Dec 16, 2014, 7:11 AM John D. Ament <[email protected]> > > wrote: > > > > > >> Hi, > > >> > > >> In the portal app, there's a dependency on Highcharts 2.3.5. As best > > as I > > >> can tell, this is a commercial license. http://www.highcharts.com/ > > >> > > >> Can you confirm this? > > >> > > >> John > > >> > > > > >
