search

himynameisdave / svelte-frappe-charts

📈 Svelte bindings for frappe-charts.
https://frappe.io/charts
MIT License
306 stars 16 forks source link

Switch to TypeScript #57

Closed youknowedo closed 9 months ago

youknowedo commented 1 year ago

Added TypeScript and annotations for a better developer experience for those who like to have everything typed. "frappe-charts" isn't typed either so most of the types in the .d.ts are option types for that package.

Closes #57

Peter-Barrett commented 1 year ago

@himynameisdave it would be great if you could merge this in. Thanks.

silvestrevivo commented 1 year ago

Any updates about this?

Peter-Barrett commented 1 year ago

For the moment adding // @ts-nocheck above the import is doing the job for me but it would be great to have this merged @himynameisdave .

himynameisdave commented 1 year ago

Ping @fenwikk, let me know if there's anything I can do to help get this over the finish line.

himynameisdave commented 1 year ago

Ping @fenwikk, sorry to be annoying here (as I really appreciate the work you've already done here), but if you're unable to address the comments I'd be happy to wrap things up here, just let me know 🙂

SemmelJochen commented 1 year ago

any news on this? Can I help somehow?

tmtygnz commented 12 months ago

Any news on this?

youknowedo commented 12 months ago

sorry guys just been busy ill fix whatever needs fixing