From b56ce56e26387d988ef66bed832905857518f6dc Mon Sep 17 00:00:00 2001
From: John Cardinal <support@ayanova.com>
Date: Fri, 11 Sep 2020 21:32:49 +0000
Subject: [PATCH]

---
 ayanova/devdocs/todo.txt             |  9 +++++----
 ayanova/src/views/ay-report-edit.vue | 27 ---------------------------
 2 files changed, 5 insertions(+), 31 deletions(-)

diff --git a/ayanova/devdocs/todo.txt b/ayanova/devdocs/todo.txt
index ebe83280..6f591a5d 100644
--- a/ayanova/devdocs/todo.txt
+++ b/ayanova/devdocs/todo.txt
@@ -3,9 +3,7 @@
 @@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
 
 
-todo: reports need to know the custom field configuration for each type along with the data in teh report meta data I guess or something else maybe
-	consumable by a function in ay-report.js which will then produce a block of localized json
-	(maybe an option for non localized as well or both in same?)
+todo: convert custom field data to usable json from the string in getReportData
 	
 todo: report add "HasLogo" or "HasSmallLogo", "HasMediumLogo" to server meta data then leverage it in the logo helper?
 
@@ -28,7 +26,10 @@ todo: (make a priority 2 case?) Will this be a thing or keep as a case for futur
 todo: export from grid to xls, csv etc.
 	I'm thinking perhaps also a secondary ability to make report templates that are actually intended for export to xls or whatever as an additional feature
 	but the easiest thing for people would be a straight export of grid data from grid, perhaps using the reportdata system since it pulls in all the bits needed for full data rather than just id links
-	
+
+Is the report list filtered by roles?  It should not present reports that a user is not permitted to use by role.
+
+Add sanitize html for Marked markdown helper as in Wiki control 	
 
 
 TODO:  Report confirmation checklist
diff --git a/ayanova/src/views/ay-report-edit.vue b/ayanova/src/views/ay-report-edit.vue
index 14548d16..47c30524 100644
--- a/ayanova/src/views/ay-report-edit.vue
+++ b/ayanova/src/views/ay-report-edit.vue
@@ -123,37 +123,10 @@
 
 import * as monaco from "monaco-editor";
 
-//jsreport demo
-//https://playground.jsreport.net/w/admin/hBfqC7af
-//jsreport text editor source
-//https://github.com/jsreport/jsreport-studio/blob/master/src/components/Editor/TextEditor.js
-
 //Monaco editor info page with links
 //https://microsoft.github.io/monaco-editor/
 
-//vue-monaco component source
-//https://github.com/egoist/vue-monaco/blob/master/src/MonacoEditor.js
 
-/*TODO:
-
-
-Is the report list filtered by roles?  It should not present reports that a user is not permitted to use by role.
-
-Add sanitize html for Marked markdown helper as in Wiki control 
-
-
-
-FUTURE:
-
-Handlebars.logger.log is a thing and may contain useful diagnostic info
-
-pre-scan the template for validity? Get the correct error, right now a missing helper is just: Evaluation failed: d 
-  which is not particularly helpful
-
-When in report designer it would be useful to show the html generated by the renderer as a diagnostic assistant
-  Also the script that will be executed minus the hb and marked etc if possible
-
-*/
 const FORM_KEY = "ay-report-edit";
 let JUST_DELETED = false;
 let editor = null;