code review

packages/server/src/api/controllers/row.js

@@ -256,7 +256,9 @@ exports.search = async function(ctx) {
     }
   }
 
-  ctx.body = await linkRows.attachLinkInfo(appId, rows)
+  const table = await db.get(ctx.params.tableId)
+
+  ctx.body = await enrichRows(appId, table, rows)
 }
 
 exports.fetchTableRows = async function(ctx) {

packages/server/src/api/routes/row.js

@@ -41,6 +41,7 @@ router
   )
   .post(
     "/api/:tableId/rows/search",
+    paramResource("tableId"),
     authorized(PermissionTypes.TABLE, PermissionLevels.READ),
     rowController.search
   )

packages/standard-components/manifest.json

@@ -146,7 +146,8 @@
       {
         "type": "text",
         "label": "No Rows Message",
-        "key": "noRowsMessage"
+        "key": "noRowsMessage",
+        "defaultValue": "No Rows"
       }
     ]
   },

packages/standard-components/src/Search.svelte

@@ -16,7 +16,7 @@
   export let table = []
   export let columns = []
   export let pageSize = 50
-  export let noRowsMessage = "No Rows"
+  export let noRowsMessage
 
   let rows = []
   let loaded = false
@@ -94,9 +94,17 @@
         }}>
         Reset
       </Button>
-      <Button primary on:click={() => fetchData(table)}>Search</Button>
+      <Button
+        primary
+        on:click={() => {
+          page = 0
+          fetchData(table, page)
+        }}>
+        Search
+      </Button>
     </div>
   </div>
+  {#if loaded}
     {#if rows.length > 0}
       {#if $component.children === 0 && $builderStore.inBuilder}
         <p>Add some components too</p>
@@ -107,11 +115,12 @@
           </DataProvider>
         {/each}
       {/if}
-  {:else if loaded && $builderStore.inBuilder}
+    {:else if $builderStore.inBuilder}
       <p>Feed me some data</p>
     {:else}
       <p>{noRowsMessage}</p>
     {/if}
+  {/if}
   <div class="pagination">
     {#if page > 0}
       <Button primary on:click={previousPage}>Back</Button>
@@ -133,7 +142,6 @@
 
   .query-builder {
     padding: var(--spacing-m);
-    background: var(--background);
     border-radius: var(--border-radius-s);
   }