Repository: internetarchive/openlibrary

Issue:
Merge Request Dashboard Chat Redesign
<!-- What problem are we solving? What does the experience look like today? What are the symptoms? -->

### Evidence / Screenshot (if possible)

It seems like it may not be obvious to some librarians that pressing "enter" in a Merge Request's chat message box will submit it (since the UI hides previous messages).

Here's one thread I expanded where the contributor submitted many of the same message, maybe out of confusion.
![openlibrary org_merges_mode_closed_page_4](https://user-images.githubusercontent.com/978325/191988183-0c0d602d-a539-4a57-9f5c-a7c3383f3dcf.png)

Maybe **solved** by this redesign: https://www.figma.com/file/kOQdKg11suMVZrDSAXTYLc/Merge-Queue?node-id=0%3A1

### Relevant url?
<!-- `https://openlibrary.org/...` -->

### Steps to Reproduce
<!-- What steps caused you to find the bug? -->
1. Go to ...
2. Do ...

<!-- What actually happened after these steps? What did you expect to happen? -->
* Actual:
* Expected:

### Details

- **Logged in (Y/N)?**
- **Browser type/version?**
- **Operating system?**
- **Environment (prod/dev/local)?** prod
<!-- If not sure, put prod -->

### Proposal & Constraints
<!-- What is the proposed solution / implementation? Is there a precedent of this approach succeeding elsewhere? -->

### Related files
<!-- Files related to this issue; this is super useful for new contributors who might want to help! If you're not sure, leave this blank; a maintainer will add them. -->

### Stakeholders
<!-- @ tag stakeholders of this bug -->



Discussion:

Resolving patch:
diff --git a/openlibrary/core/edits.py b/openlibrary/core/edits.py
--- a/openlibrary/core/edits.py
+++ b/openlibrary/core/edits.py
@@ -99,7 +99,9 @@ def get_submitters(cls):
     @classmethod
     def get_reviewers(cls):
         oldb = db.get_db()
-        query = f'SELECT DISTINCT reviewer FROM {cls.TABLENAME}'
+        query = (
+            f'SELECT DISTINCT reviewer FROM {cls.TABLENAME} WHERE reviewer IS NOT NULL'
+        )
         return list(oldb.query(query))
 
     @classmethod
diff --git a/openlibrary/plugins/upstream/edits.py b/openlibrary/plugins/upstream/edits.py
--- a/openlibrary/plugins/upstream/edits.py
+++ b/openlibrary/plugins/upstream/edits.py
@@ -54,9 +54,15 @@ def GET(self):
             "reviewers": CommunityEditsQueue.get_reviewers(),
         }
 
+        librarians = {
+            'submitters': CommunityEditsQueue.get_submitters(),
+            'reviewers': CommunityEditsQueue.get_reviewers(),
+        }
+
         return render_template(
             'merge_queue/merge_queue',
             total_found,
+            librarians,
             merge_requests=merge_requests,
         )
 
@@ -191,15 +197,5 @@ def create_title(mr_type: int, olids: list[str]) -> str:
         return 'Unknown record'
 
 
-class ui_partials(delegate.page):
-    path = '/merges/partials'
-
-    def GET(self):
-        i = web.input(type=None, comment='')
-        if i.type == 'comment':
-            component = render_template('merge_queue/comment', comment_str=i.comment)
-            return delegate.RawText(component)
-
-
 def setup():
     pass


Test patch: