padding change

src/components/Card/OverviewCard.astro

@@ -4,7 +4,8 @@ import TagRow from "./TagRow.astro"
 
 const { recipe } = Astro.props;
 
-const image = (await client.getRecipeImages(recipe.id))[0]
+const headerImage = await client.collection("images").getOne(recipe.images[0])
+const image = await client.files.getRelativeURL(headerImage, headerImage.image)
 ---
 
 <div class="relative z-0 flex h-50">
@@ -18,7 +19,7 @@ const image = (await client.getRecipeImages(recipe.id))[0]
         <!-- <p id="recipe-desc" class="text-white text-[10pt]"> {recipe.description} </p> -->
 
         <div id="tag-row" class="">
-            <TagRow tags={recipe.expand.tags}/>
+            <TagRow tagIds={recipe.tags}/>
         </div>
     </div>
 

src/components/Card/TagRow.astro

@@ -1,12 +1,29 @@
 ---
-const { tags } = Astro.props
+import client from "../../data/pocketbase"
+
+interface Props {
+    tagIds: string[]
+}
+
+const { tagIds } = Astro.props
+
+const tags = tagIds && tagIds.length > 0 
+    ? await Promise.all(tagIds.map(async (tagId: string) => {
+        try {
+            const tagData = await client.collection("tags").getOne(tagId)
+            return { name: tagData.name, id: tagId }
+        } catch (error) {
+            return null
+        }
+      }))
+    : []
 ---
 
 <div class="">
     {
-        (tags ?? []).map(tag => (
+        tags.map(tag => (
             <a 
-                href={`/tags/${tag.name}`}
+                href={`/tag/${tag.id}`}
                 class="text-white bg-white/20 px-2 mr-2 mt-2 rounded-md inline-block hover:bg-white/30" 
             >
                 {tag.name}

src/components/Detail/ImageCarousel.astro

@@ -1,7 +1,17 @@
 ---
 import client from "@/data/pocketbase";
 const { class: className, recipe } = Astro.props
-const links = await client.getRecipeImages(recipe as string)
+
+async function getLink(img: string) {
+    const record = await client.collection("images").getOne(img)
+    const link = await client.files.getRelativeURL(record, record.image)
+    return link
+}
+
+// Use Promise.all to wait for all async operations to complete
+const links = await Promise.all(
+    recipe.images.map((img: string) => getLink(img))
+)
 
 ---
 

src/components/Detail/InfoView.astro

@@ -26,4 +26,4 @@ const waitTime = formatTime(re.waittime)
     {waitTime && (<p class="border-r px-2">{waitTime} Wait</p>)}
     {re.rating !== 0 && (<p class="pl-2">{re.rating}</p><p class="text-white/40 text-xs">/10</p>)}
 </div>
-<TagRow tags={re.expand.tags}/>
+<TagRow tagIds={re.tags}/>

src/components/Detail/StepIngredientSideView.astro

@@ -3,10 +3,13 @@ import client from "@/data/pocketbase";
 
 const { ingredients, class: className } = Astro.props;
 
+const ings = await Promise.all(
+    ingredients.map(async i => await client.collection("ingredients").getOne(i))
+)
 ---
 
 <div class={className}>
-    {ingredients.map(i => (
+    {ings.map(i => (
         <div class="text-sm">
             <p>• {i.quantity} {i.unit || " "}  {i.name}</p>
         </div>

src/components/Detail/StepView.astro

@@ -16,7 +16,7 @@ const { steps, class: className } = Astro.props
                 <p class="w-full md:flex-2/3 pr-1 text-left">{s.instruction}</p>
                 {s.ingredients && s.ingredients.length > 0 && (
                     <div class="w-full md:w-auto md:flex-2/5 mt-2 md:mt-0 md:ml-2 md:pl-3 md:border-l text-white/70 border-white/70">
-                        <StepIngredientSideView class="text-left" ingredients={s.expand.ingredients} />
+                        <StepIngredientSideView class="text-left" ingredients={s.ingredients} />
                     </div>
                 )}
             </div>

src/components/Header.astro

@@ -8,8 +8,7 @@
 
     <div class="ml-auto space-x-5">
         <a class="hover:underline underline-offset-4 "  href="/recipe/new">new</a>
-        <a class="hover:underline underline-offset-4" href="/recipe/import">add</a>
+        <a class="hover:underline underline-offset-4 " >tags</a>
-        <!-- <a class="hover:underline underline-offset-4" href="/tags">tags</a> -->
         <a class="hover:underline underline-offset-4 " >search</a>
     </div>
 </div>

src/data/pocketbase.ts

@@ -1,71 +1,12 @@
-import Pocketbase, { type RecordListOptions } from "pocketbase"
+import Pocketbase from "pocketbase"
-import {
-    type Recipe,
-    type Ingredient,
-    type Step,
-    type Tag,
-    Collection
-} from './schema'
 
-class APIClient {
+const client = new Pocketbase("http://localhost:4321")
-    client: Pocketbase
+client.autoCancellation(false)
 
-    constructor() {
+// Return a relative url for file instead of full path including localhost, which breaks external access
-        this.client = new Pocketbase("http://localhost:4321")
+client.files.getRelativeURL = (record: { [key: string]: any; }, filename: string, queryParams?: FileOptions | undefined) => {
-        this.client.autoCancellation(false)
+    const res = client.files.getURL(record, filename)
+    return res.substring(21)
 }
 
-    async getRecipesPage(page: number, perPage: number = 30, options: RecordListOptions) {
-        return await this.client.collection<Recipe>(Collection.RECIPES).getList(page, perPage, options)
-    }
-
-    async getAllRecipes() {
-        return await this.client.collection<Recipe>(Collection.RECIPES).getFullList({ expand: 'ingredients,tags,steps,images,steps.ingredients' })
-    }
-
-    async getRecipe(id: string) {
-        return await this.client.collection<Recipe>(Collection.RECIPES).getOne(id, { expand: 'ingredients,tags,steps,images,steps.ingredients' })
-    }
-
-    // IMAGE
-    async getImageURL(imgID: string, relative: boolean = true) {
-        const record = await this.client.collection(Collection.IMAGES).getOne(imgID)
-        const res = this.client.files.getURL(record, record.image)
-        return relative ? res.substring(21) : res
-    }
-
-    async getRecipeImages(recipeID: string) {
-        const re = await this.getRecipe(recipeID)
-        const imgIDs = re.images ?? []
-
-        const urls = Promise.all(
-            imgIDs.map(img => this.getImageURL(img))
-        )
-        return urls
-    }
-
-    async getAllTags() {
-        return await this.client.collection<Tag>(Collection.TAGS).getFullList()
-    }
-
-    async getTag(name: string) {
-        return await this.client.collection<Tag>(Collection.TAGS).getList(1, 50, { filter: `name = '${name}'` })
-    }
-
-    async getRecipesOfTag(tagName: string) {
-        // get the tag id first
-        const tagResult = await this.getTag(tagName)
-        if (tagResult.items.length === 0) {
-            return []
-        }
-        const tag = tagResult.items[0]
-
-        return await this.client.collection<Recipe>(Collection.RECIPES).getFullList({ 
-            filter: `tags ~ '${tag.id}'`,
-            expand: 'ingredients,tags,steps,images,steps.ingredients' 
-        })
-    }
-
-}
-const client = new APIClient()
 export default client;

src/data/schema.ts

@@ -1,55 +0,0 @@
-// Base PB type
-export interface BaseRecord {
-    id: string,
-    created: string,
-    updated: string
-}
-
-export interface Ingredient extends BaseRecord {
-    quantity: string,
-    unit: string,
-    name: string
-}
-
-export interface Step extends BaseRecord {
-    index: number,
-    instruction: string,
-    ingredients?: Ingredient[]
-}
-
-export interface Tag extends BaseRecord {
-    name: string
-}
-
-// not sure Image is the best type cos it might be quite heavy to get all the fields every time but
-// it is here in case it is (a good idea)
-export interface Image extends BaseRecord {
-    id: string
-    filename: string
-    url?: string
-}
-
-export interface Recipe extends BaseRecord {
-    name: string,
-    description?: string,
-    servings?: number,
-    images?: string[], // image IDs
-    ingredients: string[]
-    steps: string[],
-    tags?: string[]
-
-    expand: {
-        images?: Image[], // image IDs,
-        ingredients: Ingredient[]
-        steps: Step[],
-        tags?: Tag[]
-    }
-}
-
-export const Collection = {
-    RECIPES: 'recipes',
-    STEPS: 'steps',
-    INGREDIENTS: 'ingredients',
-    TAGS: 'tags',
-    IMAGES: 'images'
-}

src/pages/api/[...proxy].ts

@@ -5,7 +5,8 @@ import type { APIRoute } from "astro";
 const getProxyUrl = (request: Request) => {
   const proxyUrl = new URL(import.meta.env.PUBLIC_PB_URL);
   const requestUrl = new URL(request.url);
-  return new URL(requestUrl.pathname + requestUrl.search, proxyUrl);
+
+  return new URL(requestUrl.pathname, proxyUrl);
 };
 
 export const ALL: APIRoute = async ({ request }) => {

src/pages/index.astro

@@ -3,15 +3,15 @@ import PageLayout from "@/layouts/base"
 import client from "@/data/pocketbase"
 import OverviewCard from "@/components/Card/OverviewCard"
 
-const recipes = await client.getAllRecipes()
+const recipies = await client.collection("recipes").getFullList()
 ---
 
 <PageLayout>	
 	<!-- <p class="pb-2">What would you like today?</p> -->
 
-	<div class="grid gap-3 grid-cols-1 md:grid-cols-2 lg:grid-cols-4 xl:grid-cols-6">
+	<div class="grid md:gap-2 gap-3 grid-cols-2 md:grid-cols-2 lg:grid-cols-4 xl:grid-cols-8">
 		{
-			recipes.map(r => (
+			recipies.map(r => (
 				<OverviewCard recipe={r} />
 			))
 		}

src/pages/recipe/[recipeid].astro

@@ -1,6 +1,5 @@
 ---
 import client from "@/data/pocketbase";
-
 import SiteLayout from "@/layouts/base";
 import ImageCarousel from "@/components/Detail/ImageCarousel";
 import IngredientTableView from "@/components/Detail/IngredientTableView";
@@ -9,25 +8,41 @@ import InfoView from "@/components/Detail/InfoView";
 
 const { recipeid } = Astro.params;
 
-const re = await client.getRecipe(recipeid as string)
+const re = await client.collection("recipes").getOne(recipeid ?? "0");
+
+const stepIds = re.steps
+
+let steps = await Promise.all(
+    stepIds.map(async s =>
+        await client.collection("steps").getOne(s)
+    )
+)
+
+steps = steps.sort((a, b) => a.index - b.index);
+
+const ingredients = await Promise.all(
+    re.ingredients.map(async s =>
+        await client.collection("ingredients").getOne(s)
+    )
+)
 ---
 
 <SiteLayout>
     <div class="flex flex-col md:flex-row mx-auto justify-center w-full lg:max-w-3/4 xl:max-w-2/3 2xl:max-w-1/2">
         <div class="flex md:flex-1/3 flex-col mt-2 md:mt-4 sticky">
-            <ImageCarousel class="w-full" recipe={recipeid} />
+            <ImageCarousel class="w-full" recipe={re} />
             <p class="text-[28pt] font-bold leading-11 mt-2">{re.name}</p>
 
             <!-- Details -->
             <InfoView re={re} />
 
             <p class="text-[22pt] font-bold 'md:mt-4'">Ingredients</p>
-            <IngredientTableView class:list={['md:w-80', 'px-4']} ingredients={re.expand.ingredients ?? []}  />
+            <IngredientTableView class:list={['md:w-80', 'px-4']} ingredients={ingredients}  />
         </div>
         
         <div class="flex mt-4 md:flex-2/3 w-full flex-col">
             <!-- Steps -->
-            <StepView class="md:ml-3" steps={re.expand.steps ?? []} />
+            <StepView class="md:ml-3" steps={steps} />
         </div>
 
     </div>

src/pages/tags/[name].astro

@@ -1,22 +0,0 @@
----
-import SiteLayout from "@/layouts/base";
-import client from "@/data/pocketbase";
-import OverviewCard from "@/components/Card/OverviewCard";
-
-const { name } = Astro.params
-const recipes = await client.getRecipesOfTag(name) // todo redir to 404 if not found
----
-
-<SiteLayout>
-    <p class="text-xl pb-2">
-        {recipes.length} { recipes.length == 1 ? "Recipe" : "Recipes" } with: <code class="bg-white/10 p-1 text-sm rounded-lg" >{name}</code>
-    </p>
-
-    <div class="grid md:gap-2 gap-3 grid-cols-2 md:grid-cols-2 lg:grid-cols-4 xl:grid-cols-8">
-		{
-			recipes.map(r => (
-				<OverviewCard recipe={r} />
-			))
-		}
-	</div>
-</SiteLayout>

src/pages/tags/index.astro

@@ -1,24 +0,0 @@
----
-import client from "@/data/pocketbase";
-import SiteLayout from "@/layouts/base";
-
-const tags = await client.getAllTags()
-
-const countsPerTag = await Promise.all(
-    tags.map(async t => (await client.getRecipesOfTag(t.name)).length)
-)
----
-
-<SiteLayout>
-    <p class="title pb-2">
-        {tags.length} Tags
-    </p>
-    {
-        (tags ?? []).map((t, i) => (
-            // <p>{t.name} -&gt; {countsPerTag[i]} {countsPerTag[i] == 1 ? "Recipe" : "Recipes" } </p>
-            <a class="hover:underline" href={`/tags/${t.name}`}>
-                {t.name} ({countsPerTag[i]})
-            </a><br/>
-        ))
-    }
-</SiteLayout>

src/styles/global.css

@@ -8,7 +8,3 @@ html {
     @apply font-sans;
     /* font-family: 'SF Pro Display', 'Segoe UI', 'Helvetica Neue', Arial, 'Noto Sans', sans-serif; */
 }
-
-.title {
-    @apply text-2xl;
-}