From f6788a9586da9eb885173bf8e172fc8cb3714f6a Mon Sep 17 00:00:00 2001
From: Corwin Kerr <cbkerr@umich.edu>
Date: Sun, 2 Apr 2023 13:23:10 -0400
Subject: [PATCH 01/16] Enable login page with redirect to existing url

---
 signac_dashboard/dashboard.py          | 23 ++++++++++++++++-------
 signac_dashboard/templates/layout.html |  2 ++
 signac_dashboard/templates/login.html  | 25 +++++++++++++++++++++++++
 3 files changed, 43 insertions(+), 7 deletions(-)
 create mode 100644 signac_dashboard/templates/login.html

diff --git a/signac_dashboard/dashboard.py b/signac_dashboard/dashboard.py
index 827b1ed..cd33a6b 100644
--- a/signac_dashboard/dashboard.py
+++ b/signac_dashboard/dashboard.py
@@ -582,17 +582,26 @@ def page_not_found(error):
 
         @dashboard.login_manager.unauthorized_handler
         def unauthorized_handler():
-            return self._render_error("Access token is required.")
+            request_url = request.url
+            if "module" not in request_url:
+                session['redirect_url'] = request_url
+            return render_template('login.html')
 
-        @dashboard.app.route("/login")
+        @dashboard.app.route("/login", methods = ['GET', 'POST'])
         def login():
-            provided_token = request.args.get("token")
+            if request.method == 'GET':
+                provided_token = request.args.get("token")
+            elif request.method == 'POST':
+                provided_token = request.form['token']
+            else:
+                provided_token = None
             if provided_token == self.config["ACCESS_TOKEN"]:
                 user = User(provided_token)
                 flask_login.login_user(user)
-                return redirect("/")
-
-            return self._render_error("Invalid token")
+                redirect_url = session.pop('redirect_url', '/')
+                return redirect(redirect_url)
+            else:
+                return render_template('login.html')
 
         @dashboard.app.route("/favicon.ico")
         @flask_login.login_required
@@ -678,7 +687,7 @@ def _run(args):
 
             if self.config["ACCESS_TOKEN"] is not None:
                 print(
-                    f"To access this server, connect to: "
+                    f"To access this server, connect to:\n"
                     f"http://{self.config['HOST']}:{self.config['PORT']}/"
                     f"login?token={self.config['ACCESS_TOKEN']}"
                 )
diff --git a/signac_dashboard/templates/layout.html b/signac_dashboard/templates/layout.html
index 23fc27f..f37850b 100644
--- a/signac_dashboard/templates/layout.html
+++ b/signac_dashboard/templates/layout.html
@@ -118,6 +118,7 @@ <h2 class="subtitle is-4">Modules</h2>
                     </div>
                     <div class="navbar-menu" id="dashboard-menu">
                         <div class="navbar-start">
+                            {% block searchbar %}
                             <div class="navbar-item">
                                 <form action="{{ url_for('search') }}" method="GET" id="searchform">
                                     <div class="field has-addons searchbox">
@@ -132,6 +133,7 @@ <h2 class="subtitle is-4">Modules</h2>
                                     </div>
                                 </form>
                             </div>
+                            {% endblock searchbar %}
                             {% for href, id, name, icon in menu.content %}
                             <a href="{{ href | e }}" class="navbar-item is-hidden-tablet{% if id == g.active_page %} is-active{% endif %}">
                                 {{ name }}
diff --git a/signac_dashboard/templates/login.html b/signac_dashboard/templates/login.html
new file mode 100644
index 0000000..cb99922
--- /dev/null
+++ b/signac_dashboard/templates/login.html
@@ -0,0 +1,25 @@
+{% extends "layout.html" %}
+
+{% block title %}{% endblock %}
+{% block subtitle %}{% endblock %}
+{% block view_options %}{% endblock %}
+{% block module_selector %}{% endblock %}
+{% block searchbar %}{% endblock %}
+
+{% block panels %}
+
+<div class="container column is-8">
+  <form method="POST" action="/login" class="box">
+    <h4 class="title">Logged out</h4>
+    <input class="block input is-medium" type="password" id="token" name="token" placeholder="Token">
+    <button type="submit" class="block button is-primary">Submit</button>
+    <p>
+      Token authentication is enabled. Copy the token from the command line where dashboard was run.
+    </p>
+    <p>
+      <a href="https://docs.signac.io/projects/dashboard/en/latest/security.html" target="_blank">Read more about security in dashboard.</a>
+    </p>
+  </form>
+</div>
+
+{% endblock panels %}

From 152ac5f5759a8e4b5b13d5c0990269432f3433b0 Mon Sep 17 00:00:00 2001
From: "pre-commit-ci[bot]"
 <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Date: Sun, 2 Apr 2023 17:31:48 +0000
Subject: [PATCH 02/16] [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci
---
 signac_dashboard/dashboard.py | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/signac_dashboard/dashboard.py b/signac_dashboard/dashboard.py
index cd33a6b..f0f19c4 100644
--- a/signac_dashboard/dashboard.py
+++ b/signac_dashboard/dashboard.py
@@ -584,24 +584,24 @@ def page_not_found(error):
         def unauthorized_handler():
             request_url = request.url
             if "module" not in request_url:
-                session['redirect_url'] = request_url
-            return render_template('login.html')
+                session["redirect_url"] = request_url
+            return render_template("login.html")
 
-        @dashboard.app.route("/login", methods = ['GET', 'POST'])
+        @dashboard.app.route("/login", methods=["GET", "POST"])
         def login():
-            if request.method == 'GET':
+            if request.method == "GET":
                 provided_token = request.args.get("token")
-            elif request.method == 'POST':
-                provided_token = request.form['token']
+            elif request.method == "POST":
+                provided_token = request.form["token"]
             else:
                 provided_token = None
             if provided_token == self.config["ACCESS_TOKEN"]:
                 user = User(provided_token)
                 flask_login.login_user(user)
-                redirect_url = session.pop('redirect_url', '/')
+                redirect_url = session.pop("redirect_url", "/")
                 return redirect(redirect_url)
             else:
-                return render_template('login.html')
+                return render_template("login.html")
 
         @dashboard.app.route("/favicon.ico")
         @flask_login.login_required

From 2779297247fa17aec36d910e794a4f9d0f9a15a2 Mon Sep 17 00:00:00 2001
From: Corwin Kerr <cbkerr@umich.edu>
Date: Sun, 2 Apr 2023 13:40:39 -0400
Subject: [PATCH 03/16] Update unit tests

---
 tests/test_dashboard.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/test_dashboard.py b/tests/test_dashboard.py
index da074ac..1114dd3 100644
--- a/tests/test_dashboard.py
+++ b/tests/test_dashboard.py
@@ -39,7 +39,7 @@ def setUp(self):
         # Test logged out content
         rv = self.test_client.get("/", follow_redirects=True)
         response = str(rv.get_data())
-        assert "Access token is required." in response
+        assert "Logged out" in response
 
         # login
         self.test_client.get("/login?token=test", follow_redirects=True)
@@ -47,7 +47,7 @@ def setUp(self):
     def test_invalid_token(self):
         rv = self.test_client.get("/login?token=error", follow_redirects=True)
         response = str(rv.get_data())
-        assert "Invalid token" in response
+        assert "Logged out" in response
 
     def test_get_project(self):
         rv = self.test_client.get("/project/", follow_redirects=True)

From 44e333ba1050787b2c18140a1468bde2294bb097 Mon Sep 17 00:00:00 2001
From: Corwin Kerr <cbkerr@umich.edu>
Date: Sun, 2 Apr 2023 13:40:45 -0400
Subject: [PATCH 04/16] Add test for logged out viewing a job

---
 tests/test_dashboard.py | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/tests/test_dashboard.py b/tests/test_dashboard.py
index 1114dd3..7e7372e 100644
--- a/tests/test_dashboard.py
+++ b/tests/test_dashboard.py
@@ -41,6 +41,10 @@ def setUp(self):
         response = str(rv.get_data())
         assert "Logged out" in response
 
+        rv = self.test_client.get("/jobs/7f9fb369851609ce9cb91404549393f3")
+        response = str(rv.get_data())
+        assert "Logged out" in response
+
         # login
         self.test_client.get("/login?token=test", follow_redirects=True)
 

From 23ae39abb63dd220f5eafaf85d051e70248edaef Mon Sep 17 00:00:00 2001
From: Corwin Kerr <cbkerr@umich.edu>
Date: Sun, 2 Apr 2023 13:45:07 -0400
Subject: [PATCH 05/16] Update changelog

---
 changelog.txt | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/changelog.txt b/changelog.txt
index fdb46bb..95bfb67 100644
--- a/changelog.txt
+++ b/changelog.txt
@@ -7,6 +7,14 @@ The **signac-dashboard** package follows `semantic versioning <https://semver.or
 Version 0.5
 ===========
 
+[0.5.1] -- 2023-xx-xx
+---------------------
+
+Added
++++++
+
+- Login page that redirects back to current URL (#160, #181).
+
 [0.5.0] -- 2023-03-30
 ---------------------
 

From 4c65f75e9154c5e2a118dbfb467cf74d5c0e3b5e Mon Sep 17 00:00:00 2001
From: Corwin Kerr <cbkerr@umich.edu>
Date: Sun, 2 Apr 2023 15:37:12 -0400
Subject: [PATCH 06/16] Show error message on wrong token

---
 signac_dashboard/dashboard.py | 32 ++++++++++++++++++++++++--------
 1 file changed, 24 insertions(+), 8 deletions(-)

diff --git a/signac_dashboard/dashboard.py b/signac_dashboard/dashboard.py
index f0f19c4..27891c7 100644
--- a/signac_dashboard/dashboard.py
+++ b/signac_dashboard/dashboard.py
@@ -589,19 +589,35 @@ def unauthorized_handler():
 
         @dashboard.app.route("/login", methods=["GET", "POST"])
         def login():
-            if request.method == "GET":
-                provided_token = request.args.get("token")
-            elif request.method == "POST":
-                provided_token = request.form["token"]
-            else:
-                provided_token = None
+            redirect_url = session.pop("redirect_url", "/")
+            print("redirect url", redirect_url)
+            if flask_login.current_user.is_authenticated:
+                # in case the user goes to the login page via browser history
+                return redirect(redirect_url)
+
+            provided_token = request.args.get("token") # None if not given
+            if request.method == "POST":
+                provided_token = request.form.get("token")
+
             if provided_token == self.config["ACCESS_TOKEN"]:
                 user = User(provided_token)
                 flask_login.login_user(user)
-                redirect_url = session.pop("redirect_url", "/")
+                # logs user in and goes to page they were on
                 return redirect(redirect_url)
+
             else:
-                return render_template("login.html")
+                if provided_token is None:
+                    # First time visiting page, so don't display error.
+                    return render_template("login.html")
+                flash("Incorrect token", 'danger')
+                if request.method == "GET":
+                    return redirect("/login")
+                elif request.method == "POST":
+                    if redirect_url == '/':
+                        redirect_url = '/login'
+                    return redirect(redirect_url)
+                else:
+                    return render_template("login.html")
 
         @dashboard.app.route("/favicon.ico")
         @flask_login.login_required

From ca8b2efe27efe79da878f5a99cec4550f187637a Mon Sep 17 00:00:00 2001
From: Corwin Kerr <cbkerr@umich.edu>
Date: Sun, 2 Apr 2023 15:37:23 -0400
Subject: [PATCH 07/16] Make it easier to see and copy token and url

---
 signac_dashboard/dashboard.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/signac_dashboard/dashboard.py b/signac_dashboard/dashboard.py
index 27891c7..20b611d 100644
--- a/signac_dashboard/dashboard.py
+++ b/signac_dashboard/dashboard.py
@@ -703,9 +703,9 @@ def _run(args):
 
             if self.config["ACCESS_TOKEN"] is not None:
                 print(
-                    f"To access this server, connect to:\n"
+                    f"To access this server, connect to:\n\n"
                     f"http://{self.config['HOST']}:{self.config['PORT']}/"
-                    f"login?token={self.config['ACCESS_TOKEN']}"
+                    f"login?token={self.config['ACCESS_TOKEN']}\n"
                 )
 
             self.run()

From d495b29fdd59ca55a5f77a5e3f4277a594ebf175 Mon Sep 17 00:00:00 2001
From: Corwin Kerr <cbkerr@umich.edu>
Date: Sun, 2 Apr 2023 15:37:34 -0400
Subject: [PATCH 08/16] Remove extra dash in page title with no title

---
 signac_dashboard/templates/layout.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/signac_dashboard/templates/layout.html b/signac_dashboard/templates/layout.html
index f37850b..9eb8113 100644
--- a/signac_dashboard/templates/layout.html
+++ b/signac_dashboard/templates/layout.html
@@ -16,7 +16,7 @@
         <meta name="viewport" content="width=device-width">
         <meta name="viewport" content="initial-scale=1.0">
         <title>
-            {% block title %}Page Title{% endblock %}{% if self.subtitle() %} - {% block subtitle %}{% endblock %}{% else %} - {{ APP_NAME }}{% endif %}
+            {% block title %}Page Title{% endblock %}{% if self.subtitle() %} - {% block subtitle %}{% endblock %}{% else %}{{ APP_NAME }}{% endif %}
         </title>
         {% endblock %}
         {%- assets "jquery" %}

From ab4eafd654351f0403310fdaf308f254d6ac8de1 Mon Sep 17 00:00:00 2001
From: Corwin Kerr <cbkerr@umich.edu>
Date: Sun, 2 Apr 2023 15:48:21 -0400
Subject: [PATCH 09/16] Update unit test

---
 tests/test_dashboard.py | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/tests/test_dashboard.py b/tests/test_dashboard.py
index 7e7372e..8e1e1f1 100644
--- a/tests/test_dashboard.py
+++ b/tests/test_dashboard.py
@@ -45,13 +45,13 @@ def setUp(self):
         response = str(rv.get_data())
         assert "Logged out" in response
 
-        # login
-        self.test_client.get("/login?token=test", follow_redirects=True)
-
-    def test_invalid_token(self):
         rv = self.test_client.get("/login?token=error", follow_redirects=True)
         response = str(rv.get_data())
         assert "Logged out" in response
+        assert "Incorrect token" in response
+
+        # login
+        self.test_client.get("/login?token=test", follow_redirects=True)
 
     def test_get_project(self):
         rv = self.test_client.get("/project/", follow_redirects=True)

From d3f7b829323d317dbaa7f6f029f64d80fb762351 Mon Sep 17 00:00:00 2001
From: Corwin Kerr <cbkerr@umich.edu>
Date: Sun, 2 Apr 2023 15:48:27 -0400
Subject: [PATCH 10/16] Make token selectable with double clicking

---
 signac_dashboard/dashboard.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/signac_dashboard/dashboard.py b/signac_dashboard/dashboard.py
index 20b611d..e971116 100644
--- a/signac_dashboard/dashboard.py
+++ b/signac_dashboard/dashboard.py
@@ -124,7 +124,7 @@ def __init__(self, config={}, project=None, modules=[]):
                 f"{self.config['CARDS_PER_ROW']}."
             )
 
-        self.config.setdefault("ACCESS_TOKEN", secrets.token_urlsafe())
+        self.config.setdefault("ACCESS_TOKEN", secrets.token_hex(24))
 
         # Create and configure the Flask application
         self.app = self._create_app(self.config)

From f5ffc7c5eb73415547a956a2fddb9957c395e150 Mon Sep 17 00:00:00 2001
From: "pre-commit-ci[bot]"
 <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Date: Sun, 2 Apr 2023 19:49:29 +0000
Subject: [PATCH 11/16] [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci
---
 signac_dashboard/dashboard.py | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/signac_dashboard/dashboard.py b/signac_dashboard/dashboard.py
index e971116..372a1ab 100644
--- a/signac_dashboard/dashboard.py
+++ b/signac_dashboard/dashboard.py
@@ -595,7 +595,7 @@ def login():
                 # in case the user goes to the login page via browser history
                 return redirect(redirect_url)
 
-            provided_token = request.args.get("token") # None if not given
+            provided_token = request.args.get("token")  # None if not given
             if request.method == "POST":
                 provided_token = request.form.get("token")
 
@@ -609,12 +609,12 @@ def login():
                 if provided_token is None:
                     # First time visiting page, so don't display error.
                     return render_template("login.html")
-                flash("Incorrect token", 'danger')
+                flash("Incorrect token", "danger")
                 if request.method == "GET":
                     return redirect("/login")
                 elif request.method == "POST":
-                    if redirect_url == '/':
-                        redirect_url = '/login'
+                    if redirect_url == "/":
+                        redirect_url = "/login"
                     return redirect(redirect_url)
                 else:
                     return render_template("login.html")

From c8bb4921434eee4a22417829518c6749f9b8e09c Mon Sep 17 00:00:00 2001
From: Corwin Kerr <cbkerr@umich.edu>
Date: Mon, 3 Apr 2023 09:48:22 -0400
Subject: [PATCH 12/16] Use blueprints in Notes - draft fix

---
 signac_dashboard/modules/notes.py           | 22 ++++++++++++++++-----
 signac_dashboard/templates/cards/notes.html |  2 +-
 2 files changed, 18 insertions(+), 6 deletions(-)

diff --git a/signac_dashboard/modules/notes.py b/signac_dashboard/modules/notes.py
index 799e52b..3248fdf 100644
--- a/signac_dashboard/modules/notes.py
+++ b/signac_dashboard/modules/notes.py
@@ -2,8 +2,9 @@
 # All rights reserved.
 # This software is licensed under the BSD 3-Clause License.
 import flask_login
-from flask import abort, render_template, request
+from flask import abort, Blueprint, render_template, request
 from jinja2.exceptions import TemplateNotFound
+from markupsafe import escape
 
 from signac_dashboard.module import Module
 
@@ -39,29 +40,37 @@ def __init__(
         )
         self.key = key
 
+        self.note_blueprint = Blueprint(name = key,
+                                        import_name = key,
+                                        url_prefix = f"/module/notes/{escape(key)}",
+                                        template_folder = template)
+        print(self.note_blueprint)
+
     def get_cards(self, job):
         note_text = job.document.get(self.key, "")
         return [
             {
                 "name": self.name,
                 "content": render_template(
-                    self.template, note_text=note_text, jobid=job._id
+                    self.template, note_text=note_text, jobid=job.id, note_key=self.key
                 ),
             }
         ]
 
     def register(self, dashboard):
         # Register routes
-        @dashboard.app.route("/module/notes/update", methods=["POST"])
+        @self.note_blueprint.route("/update", methods=["POST"])
         @flask_login.login_required
         def notes_update():
             note_text = request.form.get("note_text")
+            print("key is", self.key)
+            # breakpoint()
             jobid = request.form.get("jobid")
             job = dashboard.project.open_job(id=jobid)
             job.document[self.key] = note_text
             return "Saved."
 
-        @dashboard.app.route("/module/notes/<path:filename>")
+        @self.note_blueprint.route("/<path:filename>")
         @flask_login.login_required
         def notes_asset(filename):
             path = f"notes/{filename}"
@@ -76,6 +85,9 @@ def notes_asset(filename):
             dashboard.register_module_asset(
                 {
                     "file": f"templates/notes/{asset_file}",
-                    "url": f"/module/notes/{asset_file}",
+                    "url": f"/module/notes/{self.key}/{asset_file}",
                 }
             )
+
+        # register Blueprint
+        dashboard.app.register_blueprint(self.note_blueprint)
diff --git a/signac_dashboard/templates/cards/notes.html b/signac_dashboard/templates/cards/notes.html
index 7097113..c75d6cf 100644
--- a/signac_dashboard/templates/cards/notes.html
+++ b/signac_dashboard/templates/cards/notes.html
@@ -1,4 +1,4 @@
-<form class="notes-form" method="POST" action="{{ url_for('notes_update') }}">
+<form class="notes-form" method="POST" action="{{ url_for(note_key + '.notes_update') }}">
   <input type="hidden" name="redirect" value="{{ request.url }}">
   <input type="hidden" name="jobid" value="{{ jobid }}">
   <div class="field">

From 8c0207e497db8925d8e42b9d6aa8eb1a001e3753 Mon Sep 17 00:00:00 2001
From: Corwin Kerr <cbkerr@umich.edu>
Date: Mon, 3 Apr 2023 10:38:57 -0400
Subject: [PATCH 13/16] Debugging statements

---
 signac_dashboard/modules/notes.py | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/signac_dashboard/modules/notes.py b/signac_dashboard/modules/notes.py
index 3248fdf..a34cbd6 100644
--- a/signac_dashboard/modules/notes.py
+++ b/signac_dashboard/modules/notes.py
@@ -2,7 +2,7 @@
 # All rights reserved.
 # This software is licensed under the BSD 3-Clause License.
 import flask_login
-from flask import abort, Blueprint, render_template, request
+from flask import abort, Blueprint, render_template, request, redirect, url_for
 from jinja2.exceptions import TemplateNotFound
 from markupsafe import escape
 
@@ -68,6 +68,7 @@ def notes_update():
             jobid = request.form.get("jobid")
             job = dashboard.project.open_job(id=jobid)
             job.document[self.key] = note_text
+            #return redirect(url_for("show_job", jobid=jobid))
             return "Saved."
 
         @self.note_blueprint.route("/<path:filename>")
@@ -91,3 +92,8 @@ def notes_asset(filename):
 
         # register Blueprint
         dashboard.app.register_blueprint(self.note_blueprint)
+        print(self.note_blueprint.root_path)
+        # print(dashboard.app.url_map)
+        for name, blueprint in dashboard.app.blueprints.items():
+            print(f"Blueprint: {name}, URL prefix: {blueprint.url_prefix}")
+

From 2130a3049f0ab9eb2a83f9dfcce53c43a673fafa Mon Sep 17 00:00:00 2001
From: "pre-commit-ci[bot]"
 <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Date: Mon, 3 Apr 2023 14:45:09 +0000
Subject: [PATCH 14/16] [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci
---
 signac_dashboard/modules/notes.py | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/signac_dashboard/modules/notes.py b/signac_dashboard/modules/notes.py
index a34cbd6..535ac1f 100644
--- a/signac_dashboard/modules/notes.py
+++ b/signac_dashboard/modules/notes.py
@@ -2,7 +2,7 @@
 # All rights reserved.
 # This software is licensed under the BSD 3-Clause License.
 import flask_login
-from flask import abort, Blueprint, render_template, request, redirect, url_for
+from flask import Blueprint, abort, redirect, render_template, request, url_for
 from jinja2.exceptions import TemplateNotFound
 from markupsafe import escape
 
@@ -40,10 +40,12 @@ def __init__(
         )
         self.key = key
 
-        self.note_blueprint = Blueprint(name = key,
-                                        import_name = key,
-                                        url_prefix = f"/module/notes/{escape(key)}",
-                                        template_folder = template)
+        self.note_blueprint = Blueprint(
+            name=key,
+            import_name=key,
+            url_prefix=f"/module/notes/{escape(key)}",
+            template_folder=template,
+        )
         print(self.note_blueprint)
 
     def get_cards(self, job):
@@ -68,7 +70,7 @@ def notes_update():
             jobid = request.form.get("jobid")
             job = dashboard.project.open_job(id=jobid)
             job.document[self.key] = note_text
-            #return redirect(url_for("show_job", jobid=jobid))
+            # return redirect(url_for("show_job", jobid=jobid))
             return "Saved."
 
         @self.note_blueprint.route("/<path:filename>")
@@ -96,4 +98,3 @@ def notes_asset(filename):
         # print(dashboard.app.url_map)
         for name, blueprint in dashboard.app.blueprints.items():
             print(f"Blueprint: {name}, URL prefix: {blueprint.url_prefix}")
-

From 37fcce482aba31b61d8398a03659f4ea52e66fd2 Mon Sep 17 00:00:00 2001
From: Corwin Kerr <cbkerr@umich.edu>
Date: Fri, 18 Aug 2023 11:27:05 -0400
Subject: [PATCH 15/16] Remove print debugging

---
 signac_dashboard/modules/notes.py | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/signac_dashboard/modules/notes.py b/signac_dashboard/modules/notes.py
index 535ac1f..cfed9df 100644
--- a/signac_dashboard/modules/notes.py
+++ b/signac_dashboard/modules/notes.py
@@ -46,7 +46,6 @@ def __init__(
             url_prefix=f"/module/notes/{escape(key)}",
             template_folder=template,
         )
-        print(self.note_blueprint)
 
     def get_cards(self, job):
         note_text = job.document.get(self.key, "")
@@ -65,12 +64,9 @@ def register(self, dashboard):
         @flask_login.login_required
         def notes_update():
             note_text = request.form.get("note_text")
-            print("key is", self.key)
-            # breakpoint()
             jobid = request.form.get("jobid")
             job = dashboard.project.open_job(id=jobid)
             job.document[self.key] = note_text
-            # return redirect(url_for("show_job", jobid=jobid))
             return "Saved."
 
         @self.note_blueprint.route("/<path:filename>")
@@ -94,7 +90,3 @@ def notes_asset(filename):
 
         # register Blueprint
         dashboard.app.register_blueprint(self.note_blueprint)
-        print(self.note_blueprint.root_path)
-        # print(dashboard.app.url_map)
-        for name, blueprint in dashboard.app.blueprints.items():
-            print(f"Blueprint: {name}, URL prefix: {blueprint.url_prefix}")

From e462385b03d3e42740061c57df3bea56e9a11b59 Mon Sep 17 00:00:00 2001
From: Corwin Kerr <cbkerr@umich.edu>
Date: Fri, 18 Aug 2023 12:18:08 -0400
Subject: [PATCH 16/16] Fix flake8

---
 signac_dashboard/modules/notes.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/signac_dashboard/modules/notes.py b/signac_dashboard/modules/notes.py
index cfed9df..0bf4a80 100644
--- a/signac_dashboard/modules/notes.py
+++ b/signac_dashboard/modules/notes.py
@@ -2,7 +2,7 @@
 # All rights reserved.
 # This software is licensed under the BSD 3-Clause License.
 import flask_login
-from flask import Blueprint, abort, redirect, render_template, request, url_for
+from flask import Blueprint, abort, render_template, request
 from jinja2.exceptions import TemplateNotFound
 from markupsafe import escape