add todos

Author: thomasneirynck

x-pack/legacy/plugins/maps/public/layers/fields/ems_region_field.js

@@ -8,6 +8,7 @@
 import { AbstractField } from './field';
 import { TooltipProperty } from '../tooltips/tooltip_property';
 
+//todo: rename to ESMFileField
 export class EMSRegionLayerField extends AbstractField {
   static type = 'EMS_REGION_LAYER';
 

x-pack/legacy/plugins/maps/public/layers/fields/field.js

@@ -7,6 +7,8 @@
 
 export class AbstractField {
 
+  //todo consider removing
+  //double check if we're actually using this consistently
   static FIELD_TYPE = {
     STRING: 'string',
     NUMBER: 'number',

x-pack/legacy/plugins/maps/public/layers/fields/kibana_region_field.js

@@ -7,6 +7,7 @@
 
 import { AbstractField } from './field';
 
+//todo: need to be implemented
 export class KibanaRegionField extends AbstractField {
 
   static type = 'KIBANA_REGION';

x-pack/legacy/plugins/maps/public/layers/sources/es_agg_source.js

@@ -16,6 +16,7 @@ const COUNT_PROP_NAME = 'doc_count';
 
 const AGG_DELIMITER = '_of_';
 
+//todo: extract in separate PR
 export class AbstractESAggSource extends AbstractESSource {
 
   static COUNT_PROP_LABEL = COUNT_PROP_LABEL;

x-pack/legacy/plugins/maps/public/layers/styles/vector/vector_style.js

@@ -34,7 +34,6 @@ export class VectorStyle extends AbstractStyle {
 
   static type = 'VECTOR';
 
-
   static createDescriptor(properties = {}) {
     return {
       type: VectorStyle.type,