Fix null issues in view fetch, fix edit view popover, improve handling of nullish view params

packages/builder/src/components/backend/DataTable/ViewV2DataTable.svelte

@@ -1,5 +1,5 @@
 <script>
-  import { datasources, viewsV2 } from "stores/backend"
+  import { viewsV2 } from "stores/backend"
   import { Grid } from "@budibase/frontend-core"
   import { API } from "api"
   import GridCreateEditRowModal from "components/backend/DataTable/modals/grid/GridCreateEditRowModal.svelte"
@@ -12,7 +12,6 @@
     id,
     tableId: id?.split("_").slice(0, -1).join("_"),
   }
-  $: console.log(datasource)
 
   const handleGridViewUpdate = async e => {
     viewsV2.replace(id, e.detail)

packages/builder/src/components/backend/TableNavigator/TableNavigator.svelte

@@ -60,7 +60,7 @@
           }}
           selectedBy={$userSelectedResourceMap[name]}
         >
-          <EditViewPopover view={{ name, ...table.views[name] }} />
+          <EditViewPopover {view} />
         </NavItem>
       {/each}
     {/each}

packages/builder/src/components/backend/TableNavigator/popovers/EditViewPopover.svelte

@@ -1,6 +1,5 @@
 <script>
-  import { goto } from "@roxi/routify"
+  import { views, viewsV2 } from "stores/backend"
-  import { views } from "stores/backend"
   import { cloneDeep } from "lodash/fp"
   import ConfirmDialog from "components/common/ConfirmDialog.svelte"
   import {
@@ -24,25 +23,29 @@
     const updatedView = cloneDeep(view)
     updatedView.name = updatedName
 
-    await views.save({
+    if (view.version === 2) {
-      originalName,
+      await viewsV2.save({
-      ...updatedView,
+        originalName,
-    })
+        ...updatedView,
+      })
+    } else {
+      await views.save({
+        originalName,
+        ...updatedView,
+      })
+    }
+
     notifications.success("View renamed successfully")
   }
 
   async function deleteView() {
     try {
-      const isSelected =
+      if (view.version === 2) {
-        view.tableId === $views.selected.tableId &&
+        await viewsV2.delete(view)
-        view.name === $views.selected.name
+      } else {
-
+        await views.delete(view)
-      const { tableId } = view
-      await views.delete(view)
-      notifications.success("View deleted")
-      if (isSelected) {
-        $goto(`./table/${tableId}`)
       }
+      notifications.success("View deleted")
     } catch (error) {
       notifications.error("Error deleting view")
     }

packages/builder/src/helpers/urlStateSync.js

@@ -51,7 +51,7 @@ export const syncURLToState = options => {
   let cachedRedirect = get(routify.redirect)
   let cachedPage = get(routify.page)
   let previousParamsHash = null
-  let debug = true
+  let debug = false
   const log = (...params) => debug && console.log(`[${urlParam}]`, ...params)
 
   // Navigate to a certain URL

packages/builder/src/pages/builder/app/[application]/data/view/index.svelte

@@ -1,14 +1,17 @@
 <script>
   import { onMount } from "svelte"
-  import { views } from "stores/backend"
+  import { views, viewsV2 } from "stores/backend"
   import { redirect } from "@roxi/routify"
 
   onMount(async () => {
-    const { list, selected } = $views
+    if ($viewsV2.selected) {
-    if (selected) {
+      $redirect(`./v2/${$viewsV2.selected.id}`)
-      $redirect(`./${encodeURIComponent(selected?.name)}`)
+    } else if ($viewsV2.list?.length) {
-    } else if (list?.length) {
+      $redirect(`./v2/${$viewsV2.list[0].id}`)
-      $redirect(`./${encodeURIComponent(list[0].name)}`)
+    } else if ($views.selected) {
+      $redirect(`./${encodeURIComponent($views.selected?.name)}`)
+    } else if ($views.list?.length) {
+      $redirect(`./${encodeURIComponent($views.list[0].name)}`)
     } else {
       $redirect("../")
     }

packages/frontend-core/src/components/grid/stores/rows.js

@@ -102,6 +102,11 @@ export const deriveStores = context => {
     instanceLoaded.set(false)
     loading.set(true)
 
+    // Abandon if we don't have a valid datasource
+    if (!$datasource?.tableId) {
+      return
+    }
+
     // Tick to allow other reactive logic to update stores when table ID changes
     // before proceeding. This allows us to wipe filters etc if needed.
     await tick()

packages/frontend-core/src/fetch/ViewV2Fetch.js

@@ -2,13 +2,15 @@ import DataFetch from "./DataFetch.js"
 
 export default class ViewV2Fetch extends DataFetch {
   async getSchema(datasource, definition) {
-    return definition.schema
+    return definition?.schema
   }
 
   async getDefinition(datasource) {
     try {
       const table = await this.API.fetchTableDefinition(datasource.tableId)
-      const view = Object.values(table.views).find(v => v.id === datasource.id)
+      const view = Object.values(table.views || {}).find(
+        view => view.id === datasource.id
+      )
       const { schema } = view
       return { schema }
     } catch (error) {