clean up of notes, comments, BUGs, etc.

TODO

@@ -6,12 +6,15 @@
 
     * going forward into search page (and probably all POSTs) does not work 
         don't render_template instead do a redirect to a GET of the new, or list? for del...
+            * need to catch "option changes" as POSTs to make them GETs somehow...
+                (search, files_ip POST change trigger issue)
+
             [DONE] files.py:@app.route("/view/<id>", methods=["POST"])
                 files.py:@app.route("/viewlist", methods=["POST"])
                     -- this will need a total rewrite for viewer to handle fullscreen across offset/size boundaries  so fix that instead
             job.py:@app.route("/jobs", methods=["GET", "POST"])
             job.py:@app.route("/job/<id>", methods=["GET","POST"])
-                -- these ned to store 'job prefs' somewhere...
+                -- these need to store 'job prefs' somewhere...
             files.py:@app.route("/fix_dups", methods=["POST"])
 
     * optim to not run_ai_on_* for scan, needs to make sure last run_ai_on actually ran/worked - might have failed (or in my case was marked stale and I cancelled it)

states.py

@@ -23,7 +23,7 @@ class PA_UserState(db.Model):
     fullscreen = db.Column(db.Boolean, unique=False, nullable=False )
     root = db.Column(db.String, unique=False, nullable=False )
     cwd = db.Column(db.String, unique=False, nullable=False )
-    ## for now being lazy and not doing a separate table until I settle on needed fields and when
+    ## this is non normal form, but this table only ever has a handful of rows per user, not worth the extra effort
     # only used if ptype == View
     view_eid = db.Column(db.Integer, unique=False, nullable=False )
     orig_ptype = db.Column(db.String, unique=False, nullable=False )