strongloop / strong-arc

StrongLoop Arc has been replaced by API Connect. We recommend Arc users move to the Essentials edition of API Connect. If you have questions, please email reachsl@us.ibm.com.
http://strongloop.com/
Other
114 stars 36 forks source link

upgrade d3 in arc #1533

Open anthonyettinger opened 8 years ago

anthonyettinger commented 8 years ago

upgrade d3 and all related modules in arc

anthonyettinger commented 8 years ago

need to check with @jtary whether this upgrade will break visual composer.

jtary commented 8 years ago

Should be fine. If anything breaks, it should be a fairly minor fix.

cgole commented 8 years ago

Still seeing issues - https://github.com/strongloop/strong-arc/issues/1541, https://github.com/strongloop/strong-arc/issues/1540

Could not verify api analytics because of https://github.com/strongloop/strong-arc/issues/1543

cgole commented 8 years ago

@anthonyettinger, closing this one, since we have separate issues for metrics and api analytics and these bugs existed even before the d3 update.

seanbrookes commented 8 years ago

as long as we're sure the issues pre-date the library update

On Thu, Sep 17, 2015 at 11:27 PM, Chandrika Gole notifications@github.com wrote:

@anthonyettinger https://github.com/anthonyettinger, should I close this one since we have separate issues for metrics and api analytics and these bugs existed even before the d3 update.

— Reply to this email directly or view it on GitHub https://github.com/strongloop/strong-arc/issues/1533#issuecomment-141356327 .

StrongLoop http://strongloop.com/* makes it easy to develop APIs http://strongloop.com/mobile-application-development/loopback/ in Node, plus get DevOps capabilities http://strongloop.com/node-js-performance/strongops/ like monitoring, debugging and clustering*.

anthonyettinger commented 8 years ago

Yes, if there are no other problems that arose from upgrading d3. You can close this one.

anthonyettinger commented 8 years ago

These have been downgraded to issue with charts Dennis pointed out. Sean has submitted a PR since then to address the issue....so we should try upgrading again when there is sufficient time to test.