Added sign out functionality and corrected auth check (Issue #1)

brwali · Nov 17, 2023 · b832026 · b832026
1 parent e3ce737
commit b832026
Show file tree

Hide file tree

Showing 5 changed files with 39 additions and 5 deletions.
diff --git a/src/recommenderapp/app.py b/src/recommenderapp/app.py
@@ -29,7 +29,7 @@
 
 cors = CORS(app, resources={r"/*": {"origins": "*"}})
 user = {
-    1:'None'
+    1:None
 }
 
 @app.route("/")
@@ -113,6 +113,11 @@ def createAcc():
     createAccount(g.db, data["email"], data["username"], data["password"])
     return request.data
 
+@app.route("/out", methods=["POST"])
+def signout():
+    user[1] = None
+    return request.data
+
 @app.route("/log", methods=["POST"])
 def logIn():
     data = json.loads(request.data)

diff --git a/src/recommenderapp/static/script.js b/src/recommenderapp/static/script.js
@@ -138,7 +138,7 @@ $(document).ready(function () {
 			type: "POST",
 			url: "/log",
 			dataType: "json",
-      contentType: "application/json;charset=UTF-8",
+      		contentType: "application/json;charset=UTF-8",
 			traditional: "true",
 			cache: false,
 			data: JSON.stringify(data),
@@ -175,6 +175,35 @@ $(document).ready(function () {
 		createAcouunt()
 	})
 
+
+	function signOut() {
+		data = {
+			user: 'None'
+		}
+		$.ajax({
+			type: "POST",
+			url: "/out",
+			dataType: "json",
+      		contentType: "application/json;charset=UTF-8",
+			traditional: "true",
+			cache: false,
+			data: JSON.stringify(data),
+			success: function (response) {
+			// Navigate to the search page
+			setTimeout(function () {
+			window.location.href = "/" // Replace with the actual URL of your search page
+			})
+			},
+			error: function (error) {
+
+      },
+		})
+	}
+
+	$("#signOut").click(function () {
+		signOut()
+	})
+
 	function guest() {
 		data = {
 			guest:'guest'

diff --git a/src/recommenderapp/templates/landing_page.html b/src/recommenderapp/templates/landing_page.html
@@ -16,7 +16,7 @@
   <body class="landing-page"> 
     <nav class="navbar navbar-expand-lg navbar-dark bg-dark topNavBar fixed-top" id="landingTopNav">
       <div class="container-fluid">
-        <a class="navbar-brand" href="#">PopcornPicks🍿</a>
+        <a class="navbar-brand" href="#">PopcornPicks🍿</a> <button type="button" id="signOut" onclick="signOut()" style="background-color: transparent; color:white; width: 5%;">Sign Out</button>
         <button class="navbar-toggler" type="button" data-bs-toggle="collapse" data-bs-target="#navbarSupportedContent" aria-controls="navbarSupportedContent" aria-expanded="false" aria-label="Toggle navigation">
           <span class="navbar-toggler-icon"></span>
         </button>

diff --git a/src/recommenderapp/templates/search_page.html b/src/recommenderapp/templates/search_page.html
@@ -16,7 +16,7 @@
   <body>
     <nav class="navbar navbar-expand-lg navbar-dark bg-dark topNavBar fixed-top">
       <div class="container-fluid">
-        <a class="navbar-brand" href="#">PopcornPicks🍿</a>
+        <a class="navbar-brand" href="#">PopcornPicks🍿</a> <button type="button" id="signOut" onclick="signOut()" style="background-color: transparent; color:white; width: 5%;">Sign Out</button>
         <button class="navbar-toggler" type="button" data-bs-toggle="collapse" data-bs-target="#navbarSupportedContent" aria-controls="navbarSupportedContent" aria-expanded="false" aria-label="Toggle navigation">
           <span class="navbar-toggler-icon"></span>
         </button>

diff --git a/src/recommenderapp/templates/wall.html b/src/recommenderapp/templates/wall.html
@@ -21,7 +21,7 @@
   <body>
     <nav class="navbar navbar-expand-lg navbar-dark bg-dark topNavBar fixed-top">
       <div class="container-fluid">
-        <a class="navbar-brand" href="#">PopcornPicks🍿</a>
+        <a class="navbar-brand" href="#">PopcornPicks🍿</a> <button type="button" id="signOut" onclick="signOut()" style="background-color: transparent; color:white; width: 5%;">Sign Out</button>
         <button class="navbar-toggler" type="button" data-bs-toggle="collapse" data-bs-target="#navbarSupportedContent" aria-controls="navbarSupportedContent" aria-expanded="false" aria-label="Toggle navigation">
           <span class="navbar-toggler-icon"></span>
         </button>