Merge pull request #859 from genenetwork/revert-847-feature/gnqa-search-history

Revert "Feature/gnqa search history"

3 files changed, 33 insertions, 50 deletions

diff --git a/gn2/wqflask/templates/gnqa.html b/gn2/wqflask/templates/gnqa.html
index b3bc74fd..8b50fe43 100644
--- a/gn2/wqflask/templates/gnqa.html
+++ b/gn2/wqflask/templates/gnqa.html
@@ -93,7 +93,7 @@
 	    AI Search
 	    <small>
 	      <sup>
-		<button class="search-hist-btn" hx-get="/gnqna/hist" hx-target="#swap" hx-swap="innerHTML">
+		<button class="search-hist-btn" hx-get="/gnqna/hist/" hx-target="#swap" hx-swap="innerHTML" >
 		  [Search History]
 		</button>
 	      </sup>
@@ -107,7 +107,7 @@
 	    <button class="btn btn-default btn-sm col-xs-1 col-sm-1 col-sm-offset-3"
 		    hx-post="/gnqna"
 		    hx-target="#swap"
-		    hx-swap="innerHTML"		  
+		    hx-swap="innerHTML"
 		    hx-indicator="#indicator">
 		<i class="fa fa-search fa-3x" aria-hidden="true" title="Search"></i>
 		<img id="indicator" class="htmx-indicator" src="/static/gif/loader.gif"/>
diff --git a/gn2/wqflask/templates/gnqa_answer.html b/gn2/wqflask/templates/gnqa_answer.html
index 41c1b338..0ddcfde7 100644
--- a/gn2/wqflask/templates/gnqa_answer.html
+++ b/gn2/wqflask/templates/gnqa_answer.html
@@ -3,10 +3,10 @@
     <div class="row container gnqa-answer" style="margin-bottom: 1em">
 	<p class="row lead">
 	    <mark style="font-family: 'Linux Libertine','Georgia','Times','Source Serif Pro',serif;"><b><i>{{ query }}</i></b></mark><br/>
-	    {{ answer|safe }}
+	    {{ answer }}
 	</p>
 	<div class="rating row" data-doc-id="{{query}}">
-	    <button class="btn" id="upvote"  data-toggle="tooltip" data-placement="top" title="Vote Up"><i class="fa fa-thumbs-up fa-sm fa-1x" aria-hidden="true"></i></button>
+	    <button class="btn" id="upvote" data-toggle="tooltip" data-placement="top" title="Vote Up"><i class="fa fa-thumbs-up fa-sm fa-1x" aria-hidden="true"></i></button>
             <button class="btn" id="downvote" data-toggle="tooltip" data-placement="top" title="Vote Down"><i class="fa fa-thumbs-down fa-sm fa-1x" aria-hidden="true"></i></button>
 	    <sub id="rate" class="text-info">	    
 	     </sub>
@@ -32,7 +32,7 @@
 			    </div>
 			    <div id="collapse{{reference.doc_id}}" class="panel-collapse collapse in" role="tabpanel" aria-labelledby="heading{{reference.doc_id}}">
 				<div class="panel-body">
-				    <p class="node-references">{{ reference.comboTxt|safe }}</p>
+				    <p class="node-references">{{ reference.comboTxt }}</p>
 				    <div>
 					{% if reference.pubmed %}
 					<details open>
@@ -60,7 +60,7 @@
 				</div>
 				<div id="collapse{{reference.doc_id}}" class="panel-collapse collapse" role="tabpanel" aria-labelledby="heading{{reference.doc_id}}">
 				    <div class="panel-body">
-					<p class="node-references">{{ reference.comboTxt|safe }}</p>
+					<p class="node-references">{{reference.comboTxt}}</p>
 				    <div>
 					{% if reference.pubmed %}
 					<details >
@@ -93,27 +93,20 @@
 {% block js %}
 
 <script>
-  function updateRatingHandler(target, responseObj, args){
-    let {status, response} = responseObj.xhr
-    if (status==200 && args == "upvote"){
-	  htmx.toggleClass(htmx.find('#upvote'), 'btn-success');
-	  htmx.removeClass(htmx.find("#downvote"), "btn-danger");
-      }
-      else if(status == 200 && args == "downvote") {
-          htmx.toggleClass(htmx.find('#downvote'), 'btn-danger');
-	  htmx.removeClass(htmx.find("#upvote"), "btn-success");
-      }
-      else {
-        alert(`Error occurred with status ${status} and Error ${response}` )
-}}
 var query = {{ query|tojson }};
 var answer = {{ answer|tojson }}
 var {task_id} = {{ task_id|tojson }}
-htmx.on("#upvote", "click", function(evt){
+  htmx.on("#upvote", "click", function(evt){
       vote_count = htmx.find(".btn-success") ? 0 : 1
-    htmx.ajax("POST", `/gnqna/rating/${task_id}/${vote_count}`, {target: "#rate", handler: (target,obj)=> updateRatingHandler(target,obj,"upvote"), swap:"innerHTML",values: {'query': query, 'answer': answer}})});
+      htmx.ajax("POST", `/gnqna/rating/${task_id}/${vote_count}`, {target: "#rate", swap:"innerHTML",values: {'query': query, 'answer': answer}}).then(()=>{
+	  htmx.toggleClass(htmx.find('#upvote'), 'btn-success');
+	  htmx.removeClass(htmx.find("#downvote"), "btn-danger");
+})});
   htmx.on("#downvote", "click", function(evt){
       vote_count = htmx.find(".btn-danger") ? 0 : -1
-      htmx.ajax("POST", `/gnqna/rating/${task_id}/${vote_count}`, {target: "#rate",handler: (target,obj)=> updateRatingHandler(target,obj,"downvote") , swap:"innerHTML",values: {'query': query, 'answer': answer}})});
+      htmx.ajax("POST", `/gnqna/rating/${task_id}/${vote_count}`, {target: "#rate", swap:"innerHTML",values: {'query': query, 'answer': answer}}).then(()=>{
+      htmx.toggleClass(htmx.find('#downvote'), 'btn-danger');
+      htmx.removeClass(htmx.find("#upvote"), "btn-success")
+      })});
 </script>
 {% endblock %}
diff --git a/gn2/wqflask/templates/gnqa_search_history.html b/gn2/wqflask/templates/gnqa_search_history.html
index 976fd7fd..2c07b8c0 100644
--- a/gn2/wqflask/templates/gnqa_search_history.html
+++ b/gn2/wqflask/templates/gnqa_search_history.html
@@ -1,52 +1,42 @@
-<section class="container-fluid  gnqa-copy" id="search-hist">
+<section class="container-fluid  gnqa-copy">
   <header class="row">
+
     <div class="panel panel default col-sm-6 col-sm-offset-3">
       <div class="panel panel-default">
 	<div class="panel-heading">
 	  <div>
-	    <h4 class="text-secondary" style="font-family: 'Linux Libertine','Georgia','Times','Source Serif Pro',serif;font-size:2.3rem">Your AI search History </h4>
+	    <h4 class="text-primary">You search History </h4>
 	  </div>
 	</div>
       </div>
     </div>
   </header>
   <div class="container row">
-    <div>
-      <div class="col-sm-6 col-sm-offset-3" style="margin-bottom:10px">
-	<button type="button" class="btn btn-danger" id="delete-btn">Delete Selected </button>
-      </div>
-      <div >
-      <div class="panel panel-default col-sm-6 col-sm-offset-3 ">
-      <div>
-	<ul class="list-group list-group-flush" style="overflow-y:scroll">
-    {% for item in  prev_queries %}
-    <li class="row list-group-item">
-      	<input name="" type="checkbox"   value="{{item['task_id']}}" class="col-sm-1" style="height: 20px;
-  width: 20px;">
+  <div class="panel panel-default col-sm-6 col-sm-offset-3 ">
+    {% for record in prev_queries %}
+    <div class="panel-body">
+      <div class="row">
+	<input name="" type="checkbox" value="" class="col-sm-1">
 	<div class="col-sm-10">
+	  {% for id,val in  record.items() %}
 	    <button
-	      hx-get="/gnqna/hist?query={{item['query']}}&search_term={{item['task_id']}}"
+	      hx-get="/gnqna/hist/search/{{id}}"
               hx-target="#swap"
 	      hx-swap="innerHTML"
 	      hx-trigger= "click"
 	      data-bs-toggle="tooltip"
 	      data-bs-placement="left"
-	      title="/gnqna/hist?query={{item['query']}}&search_term={{item['task_id']}}"
+	      title="/gnqna/hist/search?{{id}}"
 	      style="background:transparent;border:none;cursor:pointer"
 	      >
-	      <p class="text-info">{{item.get('query')}} </p>
+	      <b class="text-info">{{val}} </b>
 	    </button>
-     </div>
-    </li>
-    {% endfor %}
-    </ul>
+	  {% endfor %}
+	</div>
+      </div>
+      </div>
+     {% endfor %}
    </div>
   </div>
   </div>
-</section>
-<script>
-  htmx.on("#delete-btn", "click", function(evt){
-      htmx.ajax("DELETE","/gnqna/hist", {target: "#search-hist","swap" :"outerHTML",
-				      values: Array.from(htmx.findAll("input[type=checkbox]:checked"), e => e.value)}) 
-  })
-</script>
+ </section>