From f039fff630e72fa1ba513146e48a460ab6a17588 Mon Sep 17 00:00:00 2001
From: Jan Schutte <4732389+SchutteJan@users.noreply.github.com>
Date: Wed, 24 Jul 2024 23:41:51 +0200
Subject: [PATCH] feat: include google_place_id in LocationResponse

---
 backend/src/db/locations.rs     |  2 ++
 backend/src/models/locations.rs |  2 ++
 frontend/src/lib/BarItem.svelte | 45 ++++++++++++++++++++++-----------
 frontend/src/models/schemas.ts  |  1 +
 4 files changed, 35 insertions(+), 15 deletions(-)

diff --git a/backend/src/db/locations.rs b/backend/src/db/locations.rs
index 10f33c0..1b79ad5 100644
--- a/backend/src/db/locations.rs
+++ b/backend/src/db/locations.rs
@@ -20,6 +20,7 @@ pub async fn get_bars(conn: &DbConn) -> Result<Vec<LocationResponse>, Error> {
                 id,
                 name,
                 description,
+                google_place_id,
                 coordinates,
                 imageurl,
                 address_line,
@@ -53,6 +54,7 @@ pub async fn get_bars_with_visits(
                 locations::id,
                 locations::name,
                 locations::description,
+                locations::google_place_id,
                 locations::coordinates,
                 locations::imageurl,
                 locations::address_line,
diff --git a/backend/src/models/locations.rs b/backend/src/models/locations.rs
index 464b8e9..bde5b41 100644
--- a/backend/src/models/locations.rs
+++ b/backend/src/models/locations.rs
@@ -28,6 +28,7 @@ pub struct LocationResponse {
     pub id: i32,
     pub name: String,
     pub description: Option<String>,
+    pub google_place_id: Option<String>,
     pub coordinates: Point,
     pub imageurl: Option<String>,
     pub address_line: String,
@@ -89,6 +90,7 @@ impl From<&Location> for LocationResponse {
             id: l.id,
             name: l.name.clone(),
             description: l.description.clone(),
+            google_place_id: l.google_place_id.clone(),
             coordinates: l.coordinates,
             imageurl: l.imageurl.clone(),
             visited_at: None,
diff --git a/frontend/src/lib/BarItem.svelte b/frontend/src/lib/BarItem.svelte
index 2df0a29..d5f0938 100644
--- a/frontend/src/lib/BarItem.svelte
+++ b/frontend/src/lib/BarItem.svelte
@@ -29,24 +29,39 @@
 	}
 </script>
 
-<article class="bar-item">
-	<img alt={bar.name} class="bar-image" src={bar.imageurl ?? placeholder} />
-	<div class="bar-content">
-		<h3>{bar.name}</h3>
+<article>
+	<details>
+		<summary class="bar-item">
+			<img alt={bar.name} class="bar-image" src={bar.imageurl ?? placeholder} />
+			<div class="bar-content">
+				<h3>{bar.name}</h3>
 
-		<p>
-			{bar.address_line} •
-			<span class="area">{bar.area_name ? bar.area_name : 'Unknown Area'}</span>
-		</p>
+				<p>
+					{bar.address_line} •
+					<span class="area">{bar.area_name ? bar.area_name : 'Unknown Area'}</span>
+				</p>
 
-		{#if isLoggedIn}
-			{#if bar.visited_at}
-				<span data-tooltip={visitString()} class="checkmark"><Checkmark /></span>
+				{#if isLoggedIn}
+					{#if bar.visited_at}
+						<span data-tooltip={visitString()} class="checkmark"><Checkmark /></span>
+					{:else}
+						<button on:click={handleVisitBar} class="visit-button outline">Check in</button>
+					{/if}
+				{/if}
+			</div>
+		</summary>
+		<hr />
+		<p>
+			{#if bar.description}
+				{bar.description}
 			{:else}
-				<button on:click={handleVisitBar} class="visit-button outline">Check in</button>
+				<i>No description available.</i>
 			{/if}
-		{/if}
-	</div>
+		</p>
+		<a target="_blank" href="https://www.google.com/maps/place/?q=place_id:{bar.google_place_id}"
+			>Open in Maps 🡵</a
+		>
+	</details>
 </article>
 
 <style>
@@ -57,7 +72,6 @@
 	.bar-image {
 		width: 6rem;
 		height: 6rem;
-		margin: 0 var(--pico-block-spacing-horizontal);
 		border-radius: var(--pico-border-radius);
 		max-width: 100%;
 		object-fit: cover;
@@ -66,6 +80,7 @@
 
 	.bar-content {
 		flex-grow: 1;
+		margin-left: var(--pico-block-spacing-horizontal);
 	}
 
 	.visit-button {
diff --git a/frontend/src/models/schemas.ts b/frontend/src/models/schemas.ts
index 00dc79d..4d75f17 100644
--- a/frontend/src/models/schemas.ts
+++ b/frontend/src/models/schemas.ts
@@ -44,6 +44,7 @@ export interface LocationResponse {
 	area_name?: string | null
 	coordinates: Point
 	description?: string | null
+	google_place_id?: string | null
 	id: number
 	imageurl?: string | null
 	name: string