From 82f2c2354940c39b1123c554d982dc60d55f7afa Mon Sep 17 00:00:00 2001 From: wanhose Date: Thu, 10 Oct 2024 13:41:50 +0200 Subject: [PATCH] feat(browser-extension): handle new error cases when reporting --- packages/api/src/index.ts | 2 - packages/api/src/routes/v5/version.ts | 27 ------------- .../src/_locales/de/messages.json | 10 ++++- .../src/_locales/en/messages.json | 10 ++++- .../src/_locales/es/messages.json | 14 +++++-- .../src/_locales/fr/messages.json | 10 ++++- .../src/_locales/it/messages.json | 10 ++++- .../src/_locales/pl/messages.json | 10 ++++- .../src/_locales/pt_BR/messages.json | 10 ++++- .../src/_locales/pt_PT/messages.json | 10 ++++- .../src/_locales/ro/messages.json | 10 ++++- .../src/_locales/ru/messages.json | 10 ++++- packages/browser-extension/src/popup.html | 30 ++++++++++++-- .../src/scripts/background.js | 17 ++++++-- .../browser-extension/src/scripts/popup.js | 40 ++++++++++++++----- .../browser-extension/src/styles/popup.css | 6 ++- 16 files changed, 157 insertions(+), 69 deletions(-) delete mode 100644 packages/api/src/routes/v5/version.ts diff --git a/packages/api/src/index.ts b/packages/api/src/index.ts index 788b2d2..b22a2c5 100644 --- a/packages/api/src/index.ts +++ b/packages/api/src/index.ts @@ -12,7 +12,6 @@ import v4ReportRoutes from 'routes/v4/report'; import v5DataRoutes from 'routes/v5/data'; import v5IssuesRoutes from 'routes/v5/issues'; import v5ReportRoutes from 'routes/v5/report'; -import v5VersionRoutes from 'routes/v5/version'; import environment from 'services/environment'; const server = fastify({ logger: true }); @@ -42,7 +41,6 @@ server.register(v4ReportRoutes, { prefix: '/rest/v4' }); server.register(v5DataRoutes, { prefix: '/rest/v5' }); server.register(v5IssuesRoutes, { prefix: '/rest/v5' }); server.register(v5ReportRoutes, { prefix: '/rest/v5' }); -server.register(v5VersionRoutes, { prefix: '/rest/v5' }); server.listen({ host: '0.0.0.0', port: environment.port }, (error, address) => { if (error) { diff --git a/packages/api/src/routes/v5/version.ts b/packages/api/src/routes/v5/version.ts deleted file mode 100644 index f6b7b51..0000000 --- a/packages/api/src/routes/v5/version.ts +++ /dev/null @@ -1,27 +0,0 @@ -import { FastifyInstance, RouteShorthandOptions } from 'fastify'; -import fetch from 'node-fetch'; -import environment from 'services/environment'; - -export default (server: FastifyInstance, _options: RouteShorthandOptions, done: () => void) => { - server.get('/version/', async (_request, reply) => { - try { - const options = { headers: { 'Cache-Control': 'no-cache' } }; - const url = `${environment.github.files}/packages/browser-extension/src/manifest.json`; - const { version } = await (await fetch(url, options)).json(); - - reply.send({ - data: { - version, - }, - success: true, - }); - } catch (error) { - reply.send({ - errors: [error.message], - success: false, - }); - } - }); - - done(); -}; diff --git a/packages/browser-extension/src/_locales/de/messages.json b/packages/browser-extension/src/_locales/de/messages.json index 2da390c..b1a8393 100644 --- a/packages/browser-extension/src/_locales/de/messages.json +++ b/packages/browser-extension/src/_locales/de/messages.json @@ -68,10 +68,16 @@ "report_reasonInputPlaceholder": { "message": "Popup ist erschienen" }, - "report_submitExtraText": { + "report_submitErrorExtraText": { + "message": "Es scheint, dass bereits ein Problem geöffnet ist oder als 'wontfix' markiert wurde. Bitte öffnen Sie das Problem und fügen Sie gegebenenfalls weitere Informationen hinzu, indem Sie den folgenden Button verwenden." + }, + "report_submitErrorText": { + "message": "Bericht nicht gesendet?" + }, + "report_submitSuccessExtraText": { "message": "Während wir daran arbeiten, können Sie diese Website zur Ausschlussliste in den Erweiterungseinstellungen hinzufügen oder einfach die Erweiterung für diese Website deaktivieren, indem Sie auf das Erweiterungssymbol in Ihrer Browser-Symbolleiste klicken." }, - "report_submitText": { + "report_submitSuccessText": { "message": "Bericht gesendet!" }, "report_urlInputError": { diff --git a/packages/browser-extension/src/_locales/en/messages.json b/packages/browser-extension/src/_locales/en/messages.json index c4b91e7..7f01ac0 100644 --- a/packages/browser-extension/src/_locales/en/messages.json +++ b/packages/browser-extension/src/_locales/en/messages.json @@ -68,10 +68,16 @@ "report_reasonInputPlaceholder": { "message": "Popup showed up" }, - "report_submitExtraText": { + "report_submitErrorExtraText": { + "message": "It seems that there's an issue open already, or it is marked as wontfix. Please enter the issue if you want to add more information using the following button." + }, + "report_submitErrorText": { + "message": "Report not sent?" + }, + "report_submitSuccessExtraText": { "message": "While we are working on this, you can add this website to the exclusion list in the extension settings or just disable the extension for this website clicking on the extension icon in your browser toolbar" }, - "report_submitText": { + "report_submitSuccessText": { "message": "Report sent!" }, "report_urlInputError": { diff --git a/packages/browser-extension/src/_locales/es/messages.json b/packages/browser-extension/src/_locales/es/messages.json index 504d7ad..edc48c5 100644 --- a/packages/browser-extension/src/_locales/es/messages.json +++ b/packages/browser-extension/src/_locales/es/messages.json @@ -68,11 +68,17 @@ "report_reasonInputPlaceholder": { "message": "Apareció un popup" }, - "report_submitExtraText": { - "message": "Mientras trabajamos en esto, puedes agregar este sitio web a la lista de exclusión en la configuración de la extensión o simplemente deshabilitar la extensión para este sitio web haciendo clic en el ícono de la extensión en la barra de herramientas de tu navegador." + "report_submitErrorExtraText": { + "message": "Reporte no enviado?" }, - "report_submitText": { - "message": "¡Informe enviado!" + "report_submitErrorText": { + "message": "Parece que ya hay un problema abierto o está marcado como 'no se solucionará'. Por favor, ingrese al problema para agregar más información usando el siguiente botón." + }, + "report_submitSuccessExtraText": { + "message": "Mientras trabajamos en esto, puedes agregar este sitio web a la lista de exclusiones en la configuración de la extensión o simplemente deshabilitar la extensión para este sitio web haciendo clic en el icono de la extensión en la barra de herramientas de tu navegador." + }, + "report_submitSuccessText": { + "message": "Reporte enviado!" }, "report_urlInputError": { "message": "Por favor, ingrese una URL válida con no más de 1000 caracteres" diff --git a/packages/browser-extension/src/_locales/fr/messages.json b/packages/browser-extension/src/_locales/fr/messages.json index 2d17841..4d5f706 100644 --- a/packages/browser-extension/src/_locales/fr/messages.json +++ b/packages/browser-extension/src/_locales/fr/messages.json @@ -68,10 +68,16 @@ "report_reasonInputPlaceholder": { "message": "Le popup est apparu" }, - "report_submitExtraText": { + "report_submitErrorExtraText": { + "message": "Il semble qu'un problème soit déjà ouvert ou marqué comme 'ne sera pas corrigé'. Veuillez entrer dans le problème pour ajouter plus d'informations en utilisant le bouton suivant." + }, + "report_submitErrorText": { + "message": "Rapport non envoyé?" + }, + "report_submitSuccessExtraText": { "message": "Pendant que nous travaillons sur ce point, vous pouvez ajouter ce site web à la liste d'exclusion dans les paramètres de l'extension ou simplement désactiver l'extension pour ce site web en cliquant sur l'icône de l'extension dans la barre d'outils de votre navigateur." }, - "report_submitText": { + "report_submitSuccessText": { "message": "Rapport envoyé!" }, "report_urlInputError": { diff --git a/packages/browser-extension/src/_locales/it/messages.json b/packages/browser-extension/src/_locales/it/messages.json index 5e83409..9b2e968 100644 --- a/packages/browser-extension/src/_locales/it/messages.json +++ b/packages/browser-extension/src/_locales/it/messages.json @@ -68,10 +68,16 @@ "report_reasonInputPlaceholder": { "message": "Popup apparso" }, - "report_submitExtraText": { + "report_submitErrorExtraText": { + "message": "Sembra che ci sia già un problema aperto o contrassegnato come 'non risolvibile'. Per favore, entra nel problema per aggiungere ulteriori informazioni usando il pulsante seguente." + }, + "report_submitErrorText": { + "message": "Segnalazione non inviata?" + }, + "report_submitSuccessExtraText": { "message": "Mentre lavoriamo su questo, puoi aggiungere questo sito web alla lista di esclusione nelle impostazioni dell'estensione o semplicemente disattivare l'estensione per questo sito web cliccando sull'icona dell'estensione nella barra degli strumenti del browser." }, - "report_submitText": { + "report_submitSuccessText": { "message": "Rapporto inviato!" }, "report_urlInputError": { diff --git a/packages/browser-extension/src/_locales/pl/messages.json b/packages/browser-extension/src/_locales/pl/messages.json index 75612f4..50926a3 100644 --- a/packages/browser-extension/src/_locales/pl/messages.json +++ b/packages/browser-extension/src/_locales/pl/messages.json @@ -68,10 +68,16 @@ "report_reasonInputPlaceholder": { "message": "Pojawił się popup" }, - "report_submitExtraText": { + "report_submitErrorExtraText": { + "message": "Wygląda na to, że istnieje już otwarte zgłoszenie lub zostało oznaczone jako 'nie do rozwiązania'. Wprowadź zgłoszenie, aby dodać więcej informacji, używając poniższego przycisku." + }, + "report_submitErrorText": { + "message": "Raport nie został wysłany?" + }, + "report_submitSuccessExtraText": { "message": "Podczas gdy nad tym pracujemy, możesz dodać tę stronę do listy wykluczeń w ustawieniach rozszerzenia lub po prostu wyłączyć rozszerzenie dla tej strony, klikając ikonę rozszerzenia na pasku narzędzi przeglądarki." }, - "report_submitText": { + "report_submitSuccessText": { "message": "Zgłoszenie wysłane!" }, "report_urlInputError": { diff --git a/packages/browser-extension/src/_locales/pt_BR/messages.json b/packages/browser-extension/src/_locales/pt_BR/messages.json index 405044f..ed36806 100644 --- a/packages/browser-extension/src/_locales/pt_BR/messages.json +++ b/packages/browser-extension/src/_locales/pt_BR/messages.json @@ -68,10 +68,16 @@ "report_reasonInputPlaceholder": { "message": "Popup apareceu" }, - "report_submitExtraText": { + "report_submitErrorExtraText": { + "message": "Parece que já existe um problema aberto ou marcado como 'não corrigido'. Por favor, acesse o problema para adicionar mais informações usando o botão a seguir." + }, + "report_submitErrorText": { + "message": "Relatório não enviado?" + }, + "report_submitSuccessExtraText": { "message": "Enquanto estamos trabalhando nisso, você pode adicionar este site à lista de exclusão nas configurações da extensão ou simplesmente desativar a extensão para este site clicando no ícone da extensão na barra de ferramentas do navegador." }, - "report_submitText": { + "report_submitSuccessText": { "message": "Relatório enviado!" }, "report_urlInputError": { diff --git a/packages/browser-extension/src/_locales/pt_PT/messages.json b/packages/browser-extension/src/_locales/pt_PT/messages.json index b0afaa4..117e1c2 100644 --- a/packages/browser-extension/src/_locales/pt_PT/messages.json +++ b/packages/browser-extension/src/_locales/pt_PT/messages.json @@ -68,10 +68,16 @@ "report_reasonInputPlaceholder": { "message": "Popup apareceu" }, - "report_submitExtraText": { + "report_submitErrorExtraText": { + "message": "Parece que já existe um problema aberto ou marcado como 'não corrigido'. Por favor, aceda ao problema para adicionar mais informações usando o botão seguinte." + }, + "report_submitErrorText": { + "message": "Relatório não enviado?" + }, + "report_submitSuccessExtraText": { "message": "Enquanto trabalhamos nisso, pode adicionar este site à lista de exclusão nas definições da extensão ou simplesmente desativar a extensão para este site clicando no ícone da extensão na barra de ferramentas do navegador." }, - "report_submitText": { + "report_submitSuccessText": { "message": "Relatório enviado!" }, "report_urlInputError": { diff --git a/packages/browser-extension/src/_locales/ro/messages.json b/packages/browser-extension/src/_locales/ro/messages.json index ac2b767..fd91fa8 100644 --- a/packages/browser-extension/src/_locales/ro/messages.json +++ b/packages/browser-extension/src/_locales/ro/messages.json @@ -68,10 +68,16 @@ "report_reasonInputPlaceholder": { "message": "A apărut un popup" }, - "report_submitExtraText": { + "report_submitErrorExtraText": { + "message": "Se pare că există deja o problemă deschisă sau marcată ca 'nu va fi rezolvată'. Vă rugăm să intrați în problemă pentru a adăuga mai multe informații folosind butonul de mai jos." + }, + "report_submitErrorText": { + "message": "Raportul nu a fost trimis?" + }, + "report_submitSuccessExtraText": { "message": "În timp ce lucrăm la acest aspect, poți adăuga acest site web la lista de excludere din setările extensiei sau poți dezactiva extensia pentru acest site web făcând clic pe pictograma extensiei din bara de instrumente a browserului." }, - "report_submitText": { + "report_submitSuccessText": { "message": "Raport trimis!" }, "report_urlInputError": { diff --git a/packages/browser-extension/src/_locales/ru/messages.json b/packages/browser-extension/src/_locales/ru/messages.json index 528de50..a4bddf7 100644 --- a/packages/browser-extension/src/_locales/ru/messages.json +++ b/packages/browser-extension/src/_locales/ru/messages.json @@ -68,10 +68,16 @@ "report_reasonInputPlaceholder": { "message": "Появилось всплывающее окно" }, - "report_submitExtraText": { + "report_submitErrorExtraText": { + "message": "Похоже, что уже существует открытая проблема или она помечена как 'не будет исправлено'. Пожалуйста, войдите в проблему, чтобы добавить больше информации, используя следующую кнопку." + }, + "report_submitErrorText": { + "message": "Отчет не отправлен?" + }, + "report_submitSuccessExtraText": { "message": "Пока мы работаем над этим, вы можете добавить этот сайт в список исключений в настройках расширения или просто отключить расширение для этого сайта, нажав на значок расширения на панели инструментов вашего браузера." }, - "report_submitText": { + "report_submitSuccessText": { "message": "Отчет отправлен!" }, "report_urlInputError": { diff --git a/packages/browser-extension/src/popup.html b/packages/browser-extension/src/popup.html index a142cf6..53bb601 100644 --- a/packages/browser-extension/src/popup.html +++ b/packages/browser-extension/src/popup.html @@ -244,7 +244,31 @@ -