dc-js / dc.leaflet.js

DC charts using Leaflet maps
Apache License 2.0
52 stars 24 forks source link

not compatible with dc.js 4.0 #39

Closed gordonwoodhull closed 4 years ago

gordonwoodhull commented 4 years ago

dc.leaflet.js still uses the dc.js 1.0 baseChart, which was changed to baseMixin in 2.0 and removed in 4.0.

msalemwhyise commented 4 years ago

+1 .. been fighting with this issue for the past 2 days to no use. Waiting for a response from the team!

gordonwoodhull commented 4 years ago

Hi @msalemwhyise! Please report issues when you run into them. I only discovered this because I was answering a question on SO.

I’ll try to push out a fix today.

gordonwoodhull commented 4 years ago

Released as 0.5.0.

Thanks for the nudge, @msalemwhyise. It was interesting to get this working, and I'm documenting what I learned in the dc@4 upgrade guide.