frappe / charts

Simple, responsive, modern SVG Charts with zero dependencies
https://frappe.io/charts
MIT License
14.9k stars 716 forks source link

Uncaught ReferenceError: transform is not defined #362

Open piyushsinghania opened 2 years ago

piyushsinghania commented 2 years ago

Expected Behaviour

No console error messages when I hover over the donut chart.

Actual Behaviour

Getting a continuous error message stating Transform is not defined

I am using the frappe/books desktop application and facing this bug/issue on the dashboard page.

Steps to Reproduce:

1) Open the frappe/books application 2) Create a supplier and a customer 3) Create an item and prepare a purchase invoice (make sure to submit the purchase invoice) 4) Navigate to the dashboard section, where one can find the expense chart 5) Open the dev tools and switch to console 6) Hover the mouse over the chart

Here I attach a gif for your reference

frappe-chart-issue

Frappé Charts version: 1.6.1 Codepen / Codesandbox:

Here I am attaching a zip file of the Expenses.vue component which is using the frappe-charts

Expenses.vue.zip

piyushsinghania commented 2 years ago

@18alantom Any update for this? From frappe charts team? cc: @ankitsinghaniyaz

18alantom commented 2 years ago

@piyushsinghania Haven't gotten to debugging this yet.

piyushsinghania commented 2 years ago

Any one there in frappe-charts team who can help in this, or look onto this? @18alantom

18alantom commented 2 years ago

@piyushsinghania I don't think anyone actively works on it, you can give it a shot.

mohamed-amine-guerras commented 2 years ago

Having the same issue in charts inside frappe dashboard, any solution?

mohamed-amine-guerras commented 2 years ago

I had the same problem with the donut chart, and after reviewing the code, I found that there is an import missing in the DonutChart.js file, which is : import { transform } from '../utils/animation'; I added that line, rebuild the code and it worked!