From 052d3fe068adb104bb2d0d94093ccce1b36d2516 Mon Sep 17 00:00:00 2001 From: Matteo Cominetti Date: Wed, 28 Jul 2021 18:15:29 +0200 Subject: [PATCH] feat(frontend): navigation refractor --- packages/frontend/public/app.html | 2 +- .../src/components/BreadcrumbTitle.vue | 139 +++++++ .../src/components/GlobalsBuilder.vue | 2 +- packages/frontend/src/router/index.js | 155 ++++---- packages/frontend/src/views/Globals.vue | 139 ------- packages/frontend/src/views/Stream.vue | 154 -------- .../src/views/settings/SettingsWebhooks.vue | 149 -------- .../src/views/settings/StreamSettings.vue | 136 ------- .../{StreamMain.vue => stream/Activity.vue} | 96 +++-- .../views/{Commits.vue => stream/Branch.vue} | 44 +-- .../src/views/{ => stream}/Branches.vue | 36 +- .../src/views/{ => stream}/Commit.vue | 65 +--- .../frontend/src/views/stream/Globals.vue | 172 +++++++++ .../src/views/{ => stream}/Object.vue | 6 +- .../Settings.vue} | 0 packages/frontend/src/views/stream/Stream.vue | 354 ++++++++++++++++++ .../frontend/src/views/stream/Webhooks.vue | 150 ++++++++ .../server/modules/core/services/streams.js | 2 +- 18 files changed, 987 insertions(+), 814 deletions(-) create mode 100644 packages/frontend/src/components/BreadcrumbTitle.vue delete mode 100644 packages/frontend/src/views/Globals.vue delete mode 100644 packages/frontend/src/views/Stream.vue delete mode 100644 packages/frontend/src/views/settings/SettingsWebhooks.vue delete mode 100644 packages/frontend/src/views/settings/StreamSettings.vue rename packages/frontend/src/views/{StreamMain.vue => stream/Activity.vue} (84%) rename packages/frontend/src/views/{Commits.vue => stream/Branch.vue} (76%) rename packages/frontend/src/views/{ => stream}/Branches.vue (82%) rename packages/frontend/src/views/{ => stream}/Commit.vue (80%) create mode 100644 packages/frontend/src/views/stream/Globals.vue rename packages/frontend/src/views/{ => stream}/Object.vue (92%) rename packages/frontend/src/views/{settings/SettingsGeneral.vue => stream/Settings.vue} (100%) create mode 100644 packages/frontend/src/views/stream/Stream.vue create mode 100644 packages/frontend/src/views/stream/Webhooks.vue diff --git a/packages/frontend/public/app.html b/packages/frontend/public/app.html index 41df93497..b4429b3c5 100644 --- a/packages/frontend/public/app.html +++ b/packages/frontend/public/app.html @@ -14,7 +14,7 @@ <%= htmlWebpackPlugin.options.title %> - + diff --git a/packages/frontend/src/components/GlobalsBuilder.vue b/packages/frontend/src/components/GlobalsBuilder.vue index 5e63e6fdb..05a9b6b1c 100644 --- a/packages/frontend/src/components/GlobalsBuilder.vue +++ b/packages/frontend/src/components/GlobalsBuilder.vue @@ -8,8 +8,8 @@ @close="closeSaveDialog" /> - Globals + Selected commit: mdi-source-commit {{ commitMessage }} diff --git a/packages/frontend/src/router/index.js b/packages/frontend/src/router/index.js index 8f0e37220..7eeac4f0e 100644 --- a/packages/frontend/src/router/index.js +++ b/packages/frontend/src/router/index.js @@ -8,7 +8,7 @@ const routes = [ path: '/authn', name: 'Auth', redirect: '/authn/login', - component: () => import('../views/Auth.vue'), + component: () => import('@/views/Auth.vue'), children: [ { path: 'login', @@ -16,7 +16,7 @@ const routes = [ meta: { title: 'Login | Speckle' }, - component: () => import('../views/auth/Login.vue') + component: () => import('@/views/auth/Login.vue') }, { path: 'register', @@ -24,7 +24,7 @@ const routes = [ meta: { title: 'Register | Speckle' }, - component: () => import('../views/auth/Registration.vue') + component: () => import('@/views/auth/Registration.vue') }, { path: 'resetpassword', @@ -32,7 +32,7 @@ const routes = [ meta: { title: 'Register | Speckle' }, - component: () => import('../views/auth/ResetPasswordRequest.vue') + component: () => import('@/views/auth/ResetPasswordRequest.vue') }, { path: 'resetpassword/finalize', @@ -40,7 +40,7 @@ const routes = [ meta: { title: 'Register | Speckle' }, - component: () => import('../views/auth/ResetPasswordFinalization.vue') + component: () => import('@/views/auth/ResetPasswordFinalization.vue') }, { path: 'verify/:appId/:challenge', @@ -48,7 +48,7 @@ const routes = [ meta: { title: 'Authorizing App | Speckle' }, - component: () => import('../views/auth/AuthorizeApp.vue') + component: () => import('@/views/auth/AuthorizeApp.vue') } ] }, @@ -57,7 +57,7 @@ const routes = [ meta: { title: 'Home | Speckle' }, - component: () => import('../views/Frontend.vue'), + component: () => import('@/views/Frontend.vue'), children: [ { path: '', @@ -65,7 +65,7 @@ const routes = [ meta: { title: 'Home | Speckle' }, - component: () => import('../views/Timeline.vue') + component: () => import('@/views/Timeline.vue') }, { path: 'streams', @@ -73,14 +73,14 @@ const routes = [ meta: { title: 'Streams | Speckle' }, - component: () => import('../views/Streams.vue') + component: () => import('@/views/Streams.vue') }, { path: 'streams/:streamId', meta: { title: 'Stream | Speckle' }, - component: () => import('../views/Stream.vue'), + component: () => import('@/views/stream/Stream.vue'), children: [ { path: '', @@ -88,31 +88,16 @@ const routes = [ meta: { title: 'Stream | Speckle' }, - component: () => import('../views/StreamMain.vue') - }, - // { - // path: 'globals/', - // name: 'globals', - // meta: { - // title: 'Globals | Speckle' - // }, - // component: () => import('../views/Globals.vue') - // }, - { - path: 'globals/:commitId', - name: 'previous globals', - meta: { - title: 'Globals | Speckle' - }, - component: () => import('../views/Globals.vue') + component: () => import('@/views/stream/Activity.vue') }, + { path: 'branches/', name: 'branches', meta: { title: 'Branches | Speckle' }, - component: () => import('../views/Branches.vue') + component: () => import('@/views/stream/Branches.vue') }, { path: 'branches/:branchName', @@ -120,15 +105,7 @@ const routes = [ meta: { title: 'Branch | Speckle' }, - component: () => import('../views/StreamMain.vue') - }, - { - path: 'branches/:branchName/commits', - name: 'commits', - meta: { - title: 'Commits | Speckle' - }, - component: () => import('../views/Commits.vue') + component: () => import('@/views/stream/Branch.vue') }, { path: 'commits/:commitId', @@ -136,7 +113,7 @@ const routes = [ meta: { title: 'Commit | Speckle' }, - component: () => import('../views/Commit.vue') + component: () => import('@/views/stream/Commit.vue') }, { path: 'objects/:objectId', @@ -144,55 +121,55 @@ const routes = [ meta: { title: 'Object | Speckle' }, - component: () => import('../views/Object.vue') + component: () => import('@/views/stream/Object.vue') }, { path: 'settings/', name: 'settings', + meta: { + title: 'Stream Settings | Speckle' + }, props: true, - component: () => import('../views/settings/StreamSettings.vue'), + component: () => import('@/views/stream/Settings.vue') + }, + { + path: 'webhooks/', + name: 'webhooks', + meta: { + title: 'Webhooks | Speckle' + }, + props: true, + component: () => import('@/views/stream/Webhooks.vue'), children: [ { - path: 'general/', - name: 'general', - meta: { - title: 'Stream Settings | Speckle' - }, - props: true, - component: () => import('../views/settings/SettingsGeneral.vue') - }, - { - path: 'webhooks/', - name: 'webhooks', - meta: { - title: 'Webhooks | Speckle' - }, - props: true, - component: () => import('../views/settings/SettingsWebhooks.vue'), - children: [ - { - path: 'edit/:webhookId/', - name: 'edit webhook', - props: true - } - ] - }, - { - path: 'webhooks/new/', - name: 'add webhook', - props: true, - component: () => import('../views/settings/SettingsWebhooks.vue') - }, - { - path: 'globals/', - name: 'globals', - meta: { - title: 'Globals | Speckle' - }, - props: true, - component: () => import('../views/Globals.vue') + path: 'edit/:webhookId/', + name: 'edit webhook', + props: true } ] + }, + { + path: 'webhooks/new/', + name: 'add webhook', + props: true, + component: () => import('@/views/stream/Webhooks.vue') + }, + { + path: 'globals/', + name: 'globals', + meta: { + title: 'Globals | Speckle' + }, + props: true, + component: () => import('@/views/stream/Globals.vue') + }, + { + path: 'globals/:commitId', + name: 'previous globals', + meta: { + title: 'Globals | Speckle' + }, + component: () => import('@/views/stream/Globals.vue') } ] }, @@ -203,7 +180,7 @@ const routes = [ meta: { title: 'Your Profile | Speckle' }, - component: () => import('../views/Profile.vue') + component: () => import('@/views/Profile.vue') }, { path: 'profile/:userId', @@ -211,7 +188,7 @@ const routes = [ meta: { title: 'User Profile | Speckle' }, - component: () => import('../views/ProfileUser.vue') + component: () => import('@/views/ProfileUser.vue') }, { path: 'admin', @@ -222,25 +199,25 @@ const routes = [ { name: 'Admin | Overview', path: '', - component: () => import('../views/admin/AdminOverview.vue') + component: () => import('@/views/admin/AdminOverview.vue') }, { name: 'Admin | Users', path: 'users', - component: () => import('../views/admin/AdminUsers.vue') + component: () => import('@/views/admin/AdminUsers.vue') }, { name: 'Admin | Streams', path: 'streams', - component: () => import('../views/admin/AdminStreams.vue') + component: () => import('@/views/admin/AdminStreams.vue') }, { name: 'Admin | Settings', path: 'settings', - component: () => import('../views/admin/AdminSettings.vue') + component: () => import('@/views/admin/AdminSettings.vue') } ], - component: () => import('../views/admin/AdminPanel.vue') + component: () => import('@/views/admin/AdminPanel.vue') } ] }, @@ -250,7 +227,7 @@ const routes = [ meta: { title: 'Error | Speckle' }, - component: () => import('../views/Error.vue') + component: () => import('@/views/Error.vue') }, { path: '/onboarding', @@ -258,12 +235,12 @@ const routes = [ meta: { title: 'Getting Started | Speckle' }, - component: () => import('../views/GettingStartedView.vue') + component: () => import('@/views/GettingStartedView.vue') }, { path: '/embed', name: 'Embeded Viewer', - component: () => import('../views/EmbedViewer.vue') + component: () => import('@/views/EmbedViewer.vue') }, { path: '*', @@ -271,7 +248,7 @@ const routes = [ meta: { title: 'Not Found | Speckle' }, - component: () => import('../views/NotFound.vue') + component: () => import('@/views/NotFound.vue') } ] diff --git a/packages/frontend/src/views/Globals.vue b/packages/frontend/src/views/Globals.vue deleted file mode 100644 index 802852a8a..000000000 --- a/packages/frontend/src/views/Globals.vue +++ /dev/null @@ -1,139 +0,0 @@ - - - - - diff --git a/packages/frontend/src/views/Stream.vue b/packages/frontend/src/views/Stream.vue deleted file mode 100644 index bae95ac75..000000000 --- a/packages/frontend/src/views/Stream.vue +++ /dev/null @@ -1,154 +0,0 @@ - - - diff --git a/packages/frontend/src/views/settings/SettingsWebhooks.vue b/packages/frontend/src/views/settings/SettingsWebhooks.vue deleted file mode 100644 index aeae185b4..000000000 --- a/packages/frontend/src/views/settings/SettingsWebhooks.vue +++ /dev/null @@ -1,149 +0,0 @@ - - - - - diff --git a/packages/frontend/src/views/settings/StreamSettings.vue b/packages/frontend/src/views/settings/StreamSettings.vue deleted file mode 100644 index 84eb6f554..000000000 --- a/packages/frontend/src/views/settings/StreamSettings.vue +++ /dev/null @@ -1,136 +0,0 @@ - - - - - diff --git a/packages/frontend/src/views/StreamMain.vue b/packages/frontend/src/views/stream/Activity.vue similarity index 84% rename from packages/frontend/src/views/StreamMain.vue rename to packages/frontend/src/views/stream/Activity.vue index 7c2ddab51..cb37a6cec 100644 --- a/packages/frontend/src/views/StreamMain.vue +++ b/packages/frontend/src/views/stream/Activity.vue @@ -1,5 +1,43 @@ + + diff --git a/packages/frontend/src/views/Object.vue b/packages/frontend/src/views/stream/Object.vue similarity index 92% rename from packages/frontend/src/views/Object.vue rename to packages/frontend/src/views/stream/Object.vue index 3a2188f35..79ac71474 100644 --- a/packages/frontend/src/views/Object.vue +++ b/packages/frontend/src/views/stream/Object.vue @@ -43,9 +43,9 @@ + diff --git a/packages/frontend/src/views/stream/Webhooks.vue b/packages/frontend/src/views/stream/Webhooks.vue new file mode 100644 index 000000000..a23ef0dae --- /dev/null +++ b/packages/frontend/src/views/stream/Webhooks.vue @@ -0,0 +1,150 @@ + + + + + diff --git a/packages/server/modules/core/services/streams.js b/packages/server/modules/core/services/streams.js index 831cf9aa5..dffbd7737 100644 --- a/packages/server/modules/core/services/streams.js +++ b/packages/server/modules/core/services/streams.js @@ -15,7 +15,7 @@ module.exports = { let stream = { id: crs( { length: 10 } ), name: name || 'Random Stream', - description: description || 'No description provided.', + description: description || '', isPublic: isPublic !== false, updatedAt: knex.fn.now( ) }