From f7ee439ced3da72180b3cedc1467d8cfc97db36f Mon Sep 17 00:00:00 2001
From: Patrick Huck <phuck@lbl.gov>
Date: Fri, 20 Aug 2021 13:30:11 -0700
Subject: [PATCH] add TODOs

---
 mpcontribs-api/mpcontribs/api/contributions/document.py   | 1 +
 mpcontribs-api/mpcontribs/api/projects/document.py        | 2 +-
 mpcontribs-client/mpcontribs/client/__init__.py           | 2 ++
 .../mpcontribs/portal/assets/js/landingpage.js            | 8 +++++++-
 4 files changed, 11 insertions(+), 2 deletions(-)

diff --git a/mpcontribs-api/mpcontribs/api/contributions/document.py b/mpcontribs-api/mpcontribs/api/contributions/document.py
index ffa36243e..2678f87ae 100644
--- a/mpcontribs-api/mpcontribs/api/contributions/document.py
+++ b/mpcontribs-api/mpcontribs/api/contributions/document.py
@@ -287,6 +287,7 @@ def make_quantities(path, key, value):
 
     @classmethod
     def post_save(cls, sender, document, **kwargs):
+        # TODO only parts of this need to run on PUT/update
         if kwargs.get("skip"):
             return
 
diff --git a/mpcontribs-api/mpcontribs/api/projects/document.py b/mpcontribs-api/mpcontribs/api/projects/document.py
index 98fb8846f..a750c6a13 100644
--- a/mpcontribs-api/mpcontribs/api/projects/document.py
+++ b/mpcontribs-api/mpcontribs/api/projects/document.py
@@ -124,7 +124,7 @@ class Projects(Document):
     authors = StringField(
         required=True,
         help_text="comma-separated list of authors"
-        # TODO set regex to enforce format
+        # TODO change to EmbeddedDocumentListField
     )
     description = StringField(
         min_length=5,
diff --git a/mpcontribs-client/mpcontribs/client/__init__.py b/mpcontribs-client/mpcontribs/client/__init__.py
index 7958ae5ff..81bb4bda7 100644
--- a/mpcontribs-client/mpcontribs/client/__init__.py
+++ b/mpcontribs-client/mpcontribs/client/__init__.py
@@ -143,6 +143,7 @@ def grouper(n, iterable):
 
 
 def get_session():
+    # TODO add Bad Gateway 502?
     adapter_kwargs = dict(max_retries=Retry(
         total=RETRIES,
         read=RETRIES,
@@ -1436,6 +1437,7 @@ def submit_contributions(
                     )
 
                     if not ignore_dupes and dupe:
+                        # TODO add matching duplicate info to msg
                         msg = f"Duplicate in {project_name}: {contrib['identifier']} {dct['name']}"
                         raise ValueError(msg)
 
diff --git a/mpcontribs-portal/mpcontribs/portal/assets/js/landingpage.js b/mpcontribs-portal/mpcontribs/portal/assets/js/landingpage.js
index c652d0150..1c2c58c77 100644
--- a/mpcontribs-portal/mpcontribs/portal/assets/js/landingpage.js
+++ b/mpcontribs-portal/mpcontribs/portal/assets/js/landingpage.js
@@ -157,11 +157,17 @@ function load_data(dom) {
                 }
             }
         }
+        // TODO loadData clears plugins/meta caches
         dom.loadData(response.data);
         if (total_count > default_limit) {
             const height = response.data.length * rowHeight;
             dom.updateSettings({height: height});
         }
+        //console.log($(hot).get(0).scrollWidth);
+        //console.log($(hot).width());
+        // TODO only collapseAll if table horizontal scroll
+        //const collapse = hot.getPlugin("collapsibleColumns");
+        //collapse.collapseAll();
         $('#table_filter').removeClass('is-loading');
         $('#table_delete').removeClass('is-loading');
         $('[name=table]').first().removeClass("is-invisible");
@@ -440,7 +446,7 @@ $("#table_get_download").click(function() { reset_table_download(); });
 $("#toggle_collapse").change(function() {
     const collapse = hot.getPlugin("collapsibleColumns");
     if (this.checked) { collapse.collapseAll(); }
-    else { collapse.expandAll(); }
+    else { collapse.expandAll(); } // TODO
 });
 
 //if ($("#graph").length && project !== 'redox_thermo_csp') {