@geode/opengeodeweb-front 8.2.0-rc.1 → 9.0.0-rc.1

package/components/{Errors/Banner.vue → FeedBack/ErrorBanner.vue}

@@ -1,8 +1,8 @@
 <template>
   <v-banner
-    v-if="server_error"
+    v-if="feedback_store.server_error"
     elevation="2"
-    style="background-color: grey; z-index: 100"
+    style="background-color: grey; z-index: 9999"
     position="fixed"
   >
     <v-row>
@@ -33,7 +33,7 @@
           size="20"
           color="grey"
           class=".align-center"
-          @click="errors_store.delete_server_error()"
+          @click="feedback_store.delete_server_error()"
         >
           <v-icon icon="mdi-close" size="20" color="white" />
         </v-btn>
@@ -43,8 +43,7 @@
 </template>
 
 <script setup>
-  const errors_store = use_errors_store()
-  const { server_error } = storeToRefs(errors_store)
+  const feedback_store = use_feedback_store()
 
   function reload() {
     window.location.reload()

package/components/{Errors → FeedBack}/Snackers.vue

@@ -1,21 +1,21 @@
 <template>
   <v-snackbar
-    v-for="(error, index) in errors"
-    :key="index"
+    v-for="(feedback, index) in feedback_store.feedbacks"
+    :key="feedback"
     v-model="show"
     :style="{ 'margin-bottom': calc_margin(index) }"
-    color="error"
+    :color="feedback.type"
     location="bottom right"
     transition="slide-x-reverse-transition"
-    max-width="30%"
+    max-width="200px"
     height="20px"
   >
     <v-row dense class="flex-nowrap">
       <v-col cols="auto">
-        <v-tooltip location="left">
+        <v-tooltip v-if="feedback.type === 'error'" location="left">
           <span>
-            error: {{ error.code }} {{ error.name }}<br />
-            ressource: {{ error.route }}
+            error: {{ feedback.code }} {{ feedback.name }}<br />
+            ressource: {{ feedback.route }}
             <br />
           </span>
           <template #activator="{ props }">
@@ -24,16 +24,21 @@
             </v-icon>
           </template>
         </v-tooltip>
+        <v-tooltip v-else-if="feedback.type === 'success'" location="left">
+          <v-icon color="white" class="justify-right">
+            mdi-check-circle-outline
+          </v-icon>
+        </v-tooltip>
       </v-col>
       <v-col cols="9" class="text-no-wrap overflow-hidden">
         <v-tooltip location="top">
           <span>
-            {{ error.description }}
+            {{ feedback.description }}
             <br />
           </span>
           <template #activator="{ props }">
             <div v-bind="props">
-              {{ error.description }}
+              {{ feedback.description }}
             </div>
           </template>
         </v-tooltip>
@@ -44,8 +49,8 @@
           icon
           variant="flat"
           size="20"
-          color="error"
-          @click="errors_store.delete_error(index)"
+          :color="feedback.type"
+          @click="feedback_store.delete_feedback(index)"
         >
           <v-icon icon="mdi-close" size="20" color="white" />
         </v-btn>
@@ -55,9 +60,7 @@
 </template>
 
 <script setup>
-  const errors_store = use_errors_store()
-  const { errors } = storeToRefs(errors_store)
-
+  const feedback_store = use_feedback_store()
   const show = true
 
   function calc_margin(index) {

package/composables/api_fetch.js

@@ -4,7 +4,7 @@ export function api_fetch(
   { schema, params },
   { request_error_function, response_function, response_error_function } = {},
 ) {
-  const errors_store = use_errors_store()
+  const feedback_store = use_feedback_store()
   const geode_store = use_geode_store()
 
   const body = params || {}
@@ -12,7 +12,8 @@ export function api_fetch(
   const { valid, error } = validate_schema(schema, body)
 
   if (!valid) {
-    errors_store.add_error({
+    feedback_store.add_feedback({
+      type: "error",
       code: 400,
       route: schema.$id,
       name: "Bad request",
@@ -39,7 +40,8 @@ export function api_fetch(
     ...request_options,
     onRequestError({ error }) {
       geode_store.stop_request()
-      errors_store.add_error({
+      feedback_store.add_feedback({
+        type: "error",
         code: error.code,
         route: schema.$id,
         name: error.message,
@@ -59,7 +61,8 @@ export function api_fetch(
     },
     onResponseError({ response }) {
       geode_store.stop_request()
-      errors_store.add_error({
+      feedback_store.add_feedback({
+        type: "error",
         code: response.status,
         route: schema.$id,
         name: response._data.name,

package/composables/upload_file.js

@@ -2,7 +2,7 @@ export async function upload_file(
   { route, file },
   { request_error_function, response_function, response_error_function } = {},
 ) {
-  const errors_store = use_errors_store()
+  const feedback_store = use_feedback_store()
   const geode_store = use_geode_store()
   if (!(file instanceof File)) {
     throw new Error("file must be a instance of File")
@@ -22,7 +22,8 @@ export async function upload_file(
     ...request_options,
     onRequestError({ error }) {
       geode_store.stop_request()
-      errors_store.add_error({
+      feedback_store.add_feedback({
+        type: "error",
         code: error.code,
         route: route,
         name: error.message,
@@ -42,7 +43,8 @@ export async function upload_file(
     },
     onResponseError({ response }) {
       geode_store.stop_request()
-      errors_store.add_error({
+      feedback_store.add_feedback({
+        type: "error",
         code: response.status,
         route: route,
         name: response._data.name,

package/composables/viewer_call.js

@@ -2,13 +2,14 @@ export function viewer_call(
   { schema, params = {} },
   { request_error_function, response_function, response_error_function } = {},
 ) {
-  const errors_store = use_errors_store()
+  const feedback_store = use_feedback_store()
   const viewer_store = use_viewer_store()
 
   const { valid, error } = validate_schema(schema, params)
 
   if (!valid) {
-    errors_store.add_error({
+    feedback_store.add_feedback({
+      type: "error",
       code: 400,
       route: schema.route,
       name: "Bad request",
@@ -41,7 +42,8 @@ export function viewer_call(
           },
         )
         .catch((error) => {
-          errors_store.add_error({
+          feedback_store.add_feedback({
+            type: "error",
             code: error.code,
             route: schema.route,
             name: error.message,

package/package.json

@@ -35,7 +35,7 @@
   },
   "description": "OpenSource Vue/Vuetify framework for web applications",
   "type": "module",
-  "version": "8.2.0-rc.1",
+  "version": "9.0.0-rc.1",
   "main": "./nuxt.config.js",
   "dependencies": {
     "@geode/opengeodeweb-back": "4.3.0",

package/stores/feedback.js

@@ -0,0 +1,17 @@
+export const use_feedback_store = defineStore("feedback", {
+  state: () => ({
+    feedbacks: [],
+    server_error: false,
+  }),
+  actions: {
+    add_feedback(feedback) {
+      this.feedbacks.push(feedback)
+    },
+    delete_feedback(feedback_index) {
+      this.feedbacks.splice(feedback_index, 1)
+    },
+    delete_server_error() {
+      this.server_error = false
+    },
+  },
+})

package/stores/geode.js

@@ -24,7 +24,7 @@ export const use_geode_store = defineStore("geode", {
       setInterval(() => this.do_ping(), 10 * 1000)
     },
     async do_ping() {
-      const errors_store = use_errors_store()
+      const feedback_store = use_feedback_store()
       const { data } = await useFetch(`${this.base_url}/ping`, {
         method: "POST",
       })
@@ -32,7 +32,7 @@ export const use_geode_store = defineStore("geode", {
         this.is_running = true
         return
       } else {
-        errors_store.$patch({ server_error: true })
+        feedback_store.$patch({ server_error: true })
         return
       }
     },

package/stores/infra.js

@@ -63,7 +63,7 @@ export const use_infra_store = defineStore("infra", {
     async create_backend() {
       const geode_store = use_geode_store()
       const viewer_store = use_viewer_store()
-      const errors_store = use_errors_store()
+      const feedback_store = use_feedback_store()
 
       if (isElectron()) {
         await window.electronAPI.run_back(geode_store.PORT)
@@ -87,7 +87,7 @@ export const use_infra_store = defineStore("infra", {
           geode_store.$patch({ is_running: true })
           return geode_store.ping_task()
         } else {
-          errors_store.server_error = true
+          feedback_store.server_error = true
         }
       }
     },

package/test/components/{Errors → FeedBack}/ErrorsBanner.nuxt.test.js

@@ -6,41 +6,41 @@ import { createVuetify } from "vuetify"
 import * as components from "vuetify/components"
 import * as directives from "vuetify/directives"
 
-import ErrorsBanner from "@/components/Errors/Banner.vue"
+import FeedBackErrorBanner from "@/components/FeedBack/ErrorBanner.vue"
 
 const vuetify = createVuetify({
   components,
   directives,
 })
 
-describe("ErrorsBanner.vue", async () => {
+describe("FeedBackErrorBanner.vue", async () => {
   test(`Test reload`, async () => {
-    const wrapper = mount(ErrorsBanner, {
+    const wrapper = mount(FeedBackErrorBanner, {
       global: {
         plugins: [vuetify],
       },
     })
     const reload_spy = vi.spyOn(wrapper.vm, "reload")
-    const errors_store = use_errors_store()
-    await errors_store.$patch({ server_error: true })
-    expect(errors_store.server_error).toBe(true)
+    const feedback_store = use_feedback_store()
+    await feedback_store.$patch({ server_error: true })
+    expect(feedback_store.server_error).toBe(true)
     const v_btn = wrapper.findAll(".v-btn")
     await v_btn[0].trigger("click")
 
     expect(reload_spy).toHaveBeenCalledTimes(1)
   }),
     test(`Test delete error`, async () => {
-      const wrapper = mount(ErrorsBanner, {
+      const wrapper = mount(FeedBackErrorBanner, {
         global: {
           plugins: [vuetify],
         },
       })
 
-      const errors_store = use_errors_store()
-      await errors_store.$patch({ server_error: true })
-      expect(errors_store.server_error).toBe(true)
+      const feedback_store = use_feedback_store()
+      await feedback_store.$patch({ server_error: true })
+      expect(feedback_store.server_error).toBe(true)
       const v_btn = wrapper.findAll(".v-btn")
       await v_btn[1].trigger("click")
-      expect(errors_store.server_error).toBe(false)
+      expect(feedback_store.server_error).toBe(false)
     })
 })

package/test/components/{Errors/ErrorsSnackers.nuxt.test.js → FeedBack/Snackers.nuxt.test.js}

@@ -6,41 +6,45 @@ import { createVuetify } from "vuetify"
 import * as components from "vuetify/components"
 import * as directives from "vuetify/directives"
 
-import ErrorsSnackers from "@/components/Errors/Snackers.vue"
+import FeedBackSnackers from "@/components/FeedBack/Snackers.vue"
 
 const vuetify = createVuetify({
   components,
   directives,
 })
 
-describe("ErrorsSnackers.vue", async () => {
+describe("FeedBackSnackers.vue", async () => {
   test(`Test delete error`, async () => {
     const wrapper = mount(
       {
-        template: "<v-layout><ErrorsSnackers/></v-layout>",
+        template: "<v-layout><FeedBackSnackers/></v-layout>",
       },
       {
         props: {},
         global: {
           components: {
-            ErrorsSnackers,
+            FeedBackSnackers,
           },
           plugins: [vuetify],
         },
       },
     )
 
-    const errors_store = use_errors_store()
+    console.log("wrapper", wrapper)
+
+    const feedback_store = use_feedback_store()
     const error = {
+      type: "error",
       code: 404,
       route: "/test",
       name: "Test message",
       description: "Test desription",
     }
-    await errors_store.add_error(error)
-    expect(errors_store.errors.length).toBe(1)
+    await feedback_store.add_feedback(error)
+    expect(feedback_store.feedbacks.length).toBe(1)
     const v_btn = await wrapper.findComponent(components.VBtn)
+    console.log("v_btn", v_btn)
     await v_btn.trigger("click")
-    expect(errors_store.errors.length).toBe(0)
+    expect(feedback_store.feedbacks.length).toBe(0)
   })
 })

package/test/composables/api_fetch.nuxt.test.js

@@ -2,7 +2,7 @@ import { describe, expect, test, beforeEach } from "vitest"
 import { registerEndpoint } from "@nuxt/test-utils/runtime"
 
 describe("api_fetch.js", () => {
-  const errors_store = use_errors_store()
+  const feedback_store = use_feedback_store()
 
   const schema = {
     $id: "/test",
@@ -19,7 +19,7 @@ describe("api_fetch.js", () => {
   var params
 
   beforeEach(async () => {
-    await errors_store.$patch({ errors: [] })
+    await feedback_store.$patch({ feedbacks: [] })
   })
 
   test("Ajv wrong params", async () => {
@@ -35,8 +35,8 @@ describe("api_fetch.js", () => {
         "/test: data must have required property 'test'",
       )
     }
-    expect(errors_store.errors.length).toBe(1)
-    expect(errors_store.errors[0].code).toBe(400)
+    expect(feedback_store.feedbacks.length).toBe(1)
+    expect(feedback_store.feedbacks[0].code).toBe(400)
   })
 
   test("onResponse", async () => {
@@ -55,7 +55,7 @@ describe("api_fetch.js", () => {
           },
         },
       )
-      expect(errors_store.errors.length).toBe(0)
+      expect(feedback_store.feedbacks.length).toBe(0)
       expect(response_value).toBe("toto")
     }
   })
@@ -83,7 +83,7 @@ describe("api_fetch.js", () => {
         },
       },
     )
-    expect(errors_store.errors.length).toBe(1)
+    expect(feedback_store.feedbacks.length).toBe(1)
     expect(response_error_value).toBe("error")
   })
 })

package/test/composables/upload_file.nuxt.test.js

@@ -5,9 +5,9 @@ import schemas from "@geode/opengeodeweb-back/schemas.json"
 const schema = schemas.opengeodeweb_back.upload_file
 
 describe("upload_file.js", () => {
-  const errors_store = use_errors_store()
+  const feedback_store = use_feedback_store()
   beforeEach(async () => {
-    await errors_store.$patch({ errors: [] })
+    await feedback_store.$patch({ errors: [] })
   })
 
   test("Throw error", async () => {
@@ -33,7 +33,7 @@ describe("upload_file.js", () => {
         },
       },
     )
-    expect(errors_store.errors.length).toBe(0)
+    expect(feedback_store.feedbacks.length).toBe(0)
     expect(response_value).toBe("ok")
   })
 })

package/test/stores/Feedback.test.js

@@ -0,0 +1,37 @@
+import { setActivePinia, createPinia } from "pinia"
+import { use_feedback_store } from "@/stores/feedback"
+import { describe, it, expect, beforeEach } from "vitest"
+
+describe("Feedback store", () => {
+  beforeEach(() => {
+    setActivePinia(createPinia())
+  })
+
+  it("add_feedback", () => {
+    const feedback_store = use_feedback_store()
+    expect(feedback_store.feedbacks.length).toBe(0)
+    feedback_store.add_feedback({
+      type: "error",
+      code: 500,
+      route: "/test",
+      name: "test message",
+      description: "test description",
+    })
+    expect(feedback_store.feedbacks.length).toBe(1)
+  })
+
+  it("delete_feedback", () => {
+    const feedback_store = use_feedback_store()
+    expect(feedback_store.feedbacks.length).toBe(0)
+    feedback_store.delete_feedback(0)
+    expect(feedback_store.feedbacks.length).toBe(0)
+  })
+
+  it("delete_server_error", () => {
+    const feedback_store = use_feedback_store()
+    feedback_store.$patch({ server_error: true })
+    expect(feedback_store.server_error).toBe(true)
+    feedback_store.delete_server_error()
+    expect(feedback_store.server_error).toBe(false)
+  })
+})

package/test/stores/Geode.test.js

@@ -28,11 +28,11 @@ describe("Geode Store", () => {
 
   test("do_ping", async () => {
     const geode_store = use_geode_store()
-    const errors_store = use_errors_store()
+    const feedback_store = use_feedback_store()
     await geode_store.do_ping()
 
     expect(geode_store.is_running).toBe(false)
-    expect(errors_store.server_error).toBe(true)
+    expect(feedback_store.server_error).toBe(true)
 
     registerEndpoint("/ping", {
       method: "POST",

package/stores/errors.js

@@ -1,17 +0,0 @@
-export const use_errors_store = defineStore("errors", {
-  state: () => ({
-    errors: [],
-    server_error: false,
-  }),
-  actions: {
-    add_error(error_object) {
-      this.errors.push(error_object)
-    },
-    delete_error(error_index) {
-      this.errors.splice(error_index, 1)
-    },
-    delete_server_error() {
-      this.server_error = false
-    },
-  },
-})

package/test/stores/Errors.test.js

@@ -1,36 +0,0 @@
-import { setActivePinia, createPinia } from "pinia"
-import { use_errors_store } from "@/stores/errors"
-import { describe, it, expect, beforeEach } from "vitest"
-
-describe("Errors Store", () => {
-  beforeEach(() => {
-    setActivePinia(createPinia())
-  })
-
-  it("add_error", () => {
-    const errors_store = use_errors_store()
-    expect(errors_store.errors.length).toBe(0)
-    errors_store.add_error({
-      code: 500,
-      route: "/test",
-      name: "test message",
-      description: "test description",
-    })
-    expect(errors_store.errors.length).toBe(1)
-  })
-
-  it("delete_error", () => {
-    const errors_store = use_errors_store()
-    expect(errors_store.errors.length).toBe(0)
-    errors_store.delete_error(0)
-    expect(errors_store.errors.length).toBe(0)
-  })
-
-  it("delete_server_error", () => {
-    const errors_store = use_errors_store()
-    errors_store.$patch({ server_error: true })
-    expect(errors_store.server_error).toBe(true)
-    errors_store.delete_server_error()
-    expect(errors_store.server_error).toBe(false)
-  })
-})