Closed sridharan-dlt closed 3 years ago
Following up. Any comments on this?
@sridharan-dlt Requesting you to review @arpit9616 comment's and do final fixes.
Please do this ASAP so that @TekMonksGitHub can merge.
PS. @sridharan-dlt, take a pull before you do the next commit to this branch as there are merge-conflicts from your last commits.
sure @manmeetsainiDLT. Noted everything, and will work on those changes & make PR ASAP.
Closing - as there are multiple issues. Please send a new pull once fixed.
Changes
backend/apps/monkvision/apis/exportCSV.js
frontend/apps/monkvision/components/chart-box/chart-box.html
frontend/apps/monkvision/components/span-with-menu/span-with-menu.html, frontend/apps/monkvision/main.html
frontend/apps/monkvision/dashboards/dashboard2.page
frontend/apps/monkvision/components/chart-box/chart-box.mjs
Export CSV feature is based on the backend part, which purely depends on DB. To test this feature, you must need monboss.db. I tested properly from my end with local DB.