From 9dff79140de755c7d8e307a3615810248ec133f1 Mon Sep 17 00:00:00 2001
From: Adaline Simonian <adalinesimonian@gmail.com>
Date: Mon, 13 Jan 2025 10:37:53 -0800
Subject: [PATCH] refactor: address review comments

---
 components/ProfileOverview.vue                 |  2 +-
 .../pronouns/PronounsCustomGenerator.vue       |  2 +-
 locale/data.ts                                 |  1 +
 locale/nn/config.suml                          | 10 ++++------
 locale/nn/pronouns/examples.tsv                | 18 +++++++++++++++---
 locale/nn/translations.suml                    |  2 --
 locale/no/config.suml                          | 15 +++++++--------
 locale/no/pronouns/examples.tsv                | 18 +++++++++++++++---
 locale/no/translations.suml                    |  9 ---------
 server/express/pronouns.ts                     |  4 ++--
 src/classes.ts                                 |  7 +++++--
 src/data.ts                                    |  5 +++--
 12 files changed, 54 insertions(+), 39 deletions(-)

diff --git a/components/ProfileOverview.vue b/components/ProfileOverview.vue
index 827d2377a..591415283 100644
--- a/components/ProfileOverview.vue
+++ b/components/ProfileOverview.vue
@@ -1,7 +1,7 @@
 <template>
     <div class="d-md-flex justify-content-between align-items-center">
         <h4 class="my-md-0">
-            {{ $locales[locale].name }} {{ $locales[locale].extra ? `(${ $locales[locale].extra })` : '' }}
+            {{ $locales[locale].name }} {{ $locales[locale].extra ? `(${$locales[locale].extra})` : '' }}
         </h4>
         <div v-if="profile" class="d-flex gap-2">
             <LocaleLink :locale="locale" :link="`/@${username}`" class="btn btn-primary">
diff --git a/components/pronouns/PronounsCustomGenerator.vue b/components/pronouns/PronounsCustomGenerator.vue
index 068466c1a..a3fcc9d61 100644
--- a/components/pronouns/PronounsCustomGenerator.vue
+++ b/components/pronouns/PronounsCustomGenerator.vue
@@ -14,7 +14,7 @@ const { $translator: translator } = useNuxtApp();
 const exampleCategories = ExampleCategory.from(examples, config);
 const examplesByHonorific = [false, true].map((isHonorific) => {
     const examples = exampleCategories
-        .filter((exampleCategory) => !exampleCategory.comprehensive)
+        .filter((exampleCategory) => !exampleCategory.comprehensive && exampleCategory.examples.length > 0)
         .map((exampleCategory) => exampleCategory.examples[0])
         .filter((example) => example.isHonorific === isHonorific);
     return { examples, isHonorific };
diff --git a/locale/data.ts b/locale/data.ts
index 3e25ee3de..dcf62c995 100644
--- a/locale/data.ts
+++ b/locale/data.ts
@@ -28,6 +28,7 @@ export interface PronounExamplesData {
     null?: string;
     null_plural?: string;
     isHonorific?: boolean;
+    categories?: string;
 }
 
 export type NounTemplatesData = {
diff --git a/locale/nn/config.suml b/locale/nn/config.suml
index 9c6051283..ba9e119ff 100644
--- a/locale/nn/config.suml
+++ b/locale/nn/config.suml
@@ -11,14 +11,13 @@ pronouns:
     honorifics: false
     exampleCategories:
         -
-            name: 'subjektspronomen'
+            name: 'subjekt'
             morphemes: ['pronoun_subject']
         -
-            name: 'objektspronomen'
+            name: 'objekt'
             morphemes: ['pronoun_object']
         -
-            name: 'possessivpronomen'
-            morphemes: ['possessive']
+            name: 'possessiv'
     generator:
         enabled: true
         slashes: true
@@ -92,7 +91,6 @@ pronouns:
             - 'Person B brukar {/ho=ho/henne}, so når ho snakkar om person A, brukar ho «ho/henne» om henne.'
             - 'Person C brukar {/hin=hin/hins} om einannan med {/hen=hen/hens}, so når hin snakkar om person A, brukar hen anten hin/hins eller hen/hens om hen.'
     ask:
-        name: 'Spør meg'
         routes: ['spoer', 'spoer-meg']
     others: 'Andre pronomen'
 
@@ -100,7 +98,7 @@ pronunciation:
     enabled: true
     voices:
         NO:
-            language: 'nn-NO'
+            language: 'nb-NO'
             voice: 'Liv'
             engine: 'standard'
 
diff --git a/locale/nn/pronouns/examples.tsv b/locale/nn/pronouns/examples.tsv
index 38bcafbc5..8c9cdb806 100644
--- a/locale/nn/pronouns/examples.tsv
+++ b/locale/nn/pronouns/examples.tsv
@@ -1,4 +1,16 @@
-singular	null
+singular	null	categories
 Eg synest {pronoun_subject} er veldig snill.
-{'pronoun_subject} fortalde meg at huset er {possessive}.
-Eg bad {pronoun_object} om eg kunne låne blyanten {possessive}.	Eg bad {pronoun_object} om eg kunne låne blyanten til {pronoun_subject}.
+{'pronoun_subject} fortalde meg at huset er {possessive}.		possessiv
+Eg spurde {pronoun_object} om eg kunne låne blyanten {possessive}.	Eg spurde {pronoun_object} om eg kunne låne blyanten til {pronoun_object}.	possessiv
+Eg og {pronoun_subject} vil jobbe saman med prosjekta {possessive}.	Eg og {pronoun_subject} vil jobbe saman med {pronoun_subject} sine prosjekt.	possessiv
+Kan eg ringje {pronoun_object} etter at {pronoun_subject} har ete middagen sin?
+Har du sett {pronoun_object}? Eg må levere boka {possessive} tilbake til {pronoun_subject}.	Har du sett {pronoun_object}? Eg må levere {pronoun_subject} si bok tilbake.	possessiv
+{'pronoun_subject} mista lommeboka si i går.
+Eg gav {pronoun_object} planta {possessive} fordi {pronoun_subject} like gjerde den.	Eg gav {pronoun_object} {possessive} plante fordi {pronoun_subject} like gjerde den.	possessiv
+Domaren sa at {pronoun_object} viser mykje respekt.
+Etter forelesninga takka {pronoun_subject} læraren for hjelpa.
+Kven meinte du at {pronoun_subject} ville ha med på turen?
+Vennene til {pronoun_object} hjelpte {pronoun_object} med å jobbe på bilen.		possessiv
+Kan du overlevere {possessive} bok til {pronoun_object}?		possessiv
+Mora til {pronoun_object} er meir stolt av {possessive} prestasjon.	{pronoun_subject} si mor er meir stolt av {possessive} prestasjon.	possessiv
+{'pronoun_subject} grét då {pronoun_subject} sa at innsatsen {possessive} ikkje var nok.	{pronoun_subject} grét då {pronoun_subject} sa at {possessive} sin innsats ikkje var nok.	possessiv
diff --git a/locale/nn/translations.suml b/locale/nn/translations.suml
index c81188d8e..ede360cd8 100644
--- a/locale/nn/translations.suml
+++ b/locale/nn/translations.suml
@@ -295,7 +295,6 @@ terminology:
             name: 'Fordomer'
             description: 'Omgrep relatert til hat og diskriminering mot skeive personar'
 
-
 faq:
     header: 'Spørsmål'
     headerLong: 'Ofte stilte spørsmål'
@@ -746,7 +745,6 @@ contact:
         design:
             header: 'Rettleiingar og resursar for design'
 
-
 workshops:
     header: 'Opplæring'
     headerLong: 'Opplæring'
diff --git a/locale/no/config.suml b/locale/no/config.suml
index 254ad13b9..0ca1b980f 100644
--- a/locale/no/config.suml
+++ b/locale/no/config.suml
@@ -11,14 +11,13 @@ pronouns:
     honorifics: false
     exampleCategories:
         -
-            name: 'subjektspronomen'
+            name: 'subjekt'
             morphemes: ['pronoun_subject']
         -
-            name: 'objektspronomen'
+            name: 'objekt'
             morphemes: ['pronoun_object']
         -
-            name: 'possessivpronomen'
-            morphemes: ['possessive']
+            name: 'possessiv'
     generator:
         enabled: true
         slashes: true
@@ -71,7 +70,7 @@ pronouns:
                 normative: true
                 examples:
                     - ['Kjøpte du ei gave til Tex for henne?', 'Kjøpte du ei gave til Tex?']
-                    - ['Ja, jeg kjøpte den til han. Jeg gjev den til han i morgon.', 'Ja, jeg kjøpte den. Jeg gjev den i morgen.']
+                    - ['Ja, jeg kjøpte den til han. Jeg gir den til han i morgen.', 'Ja, jeg kjøpte den. Jeg gir den i morgen.']
     emoji:
         description: 'Emojiself-pronomen'
         history: '{https://www.lgbtqia.wiki/wiki/Emojiself_Pronouns=Emojiself} pronomen er mente for nettkommunikasjon og er ikke tenkt å bli uttalt.'
@@ -91,7 +90,6 @@ pronouns:
             - 'Person B bruker {/hun=hun/henne}, så når hun snakker om person A, bruker hun «hun/henne» om henne.'
             - 'Person C bruker {/hin=hin/hins} om hverandre med {/hen=hen/hens}, så når hin snakker om person A, bruker hen enten hin/hins eller hen/hens om hen.'
     ask:
-        name: 'Spør meg'
         routes: ['spoer', 'spoer-meg']
     others: 'Andre pronomen'
 
@@ -142,10 +140,10 @@ terminology:
             key: 'kjønn'
             text: 'kjønn'
         -
-            key: 'kjønnsuttrykk'
+            key: 'kjønns uttrykk'
             text: 'kjønnsuttrykk'
         -
-            key: 'forholdsmodell'
+            key: 'forholds modell'
             text: 'forholdsmodell'
         -
             key: 'språk'
@@ -233,6 +231,7 @@ census:
     enabled: false
 
 redirects:
+    - { from: '^/faq', to: '/oss' }
     - { from: '^/ordbok/terminologi', to: '/terminologi' }
 
 api: ~
diff --git a/locale/no/pronouns/examples.tsv b/locale/no/pronouns/examples.tsv
index 7c0d2a129..f42e15943 100644
--- a/locale/no/pronouns/examples.tsv
+++ b/locale/no/pronouns/examples.tsv
@@ -1,4 +1,16 @@
-singular	null
+singular	null	categories
 Jeg synes {pronoun_subject} er veldig snill.
-{'pronoun_subject} fortalte meg at huset er {possessive}.
-Jeg spurte {pronoun_object} om jeg kunne låne blyanten {possessive}.	Jeg spurte {pronoun_object} om jeg kunne låne {possessive} blyant.
+{'pronoun_subject} fortalte meg at huset er {possessive}.		possessiv
+Jeg spurte {pronoun_object} om jeg kunne låne blyanten {possessive}.	Jeg spurte {pronoun_object} om jeg kunne låne {possessive} blyant.	possessiv
+Jeg og {pronoun_subject} vil jobbe sammen med {possessive} prosjekter.		possessiv
+Kan jeg ringe {pronoun_object} etter {pronoun_subject} har spist middagen sin?
+Har du sett {pronoun_object}? Jeg må levere boka {possessive} tilbake til {pronoun_subject}.	Har du sett {pronoun_object}? Jeg må levere {pronoun_subject} si bok tilbake.	possessiv
+{'pronoun_subject} mista lommeboka si i går.
+Jeg ga {pronoun_object} planta {possessive} fordi {pronoun_subject} likte den.	Jeg ga {pronoun_object} {possessive} plante fordi {pronoun_subject} likte den.	possessiv
+Dommeren sa at {pronoun_object} viser mye respekt.
+Etter forelesninga takka {pronoun_subject} læreren for hjelpa.
+Hvem mente du at {pronoun_subject} ville ha med på turen?
+Vennene til {pronoun_object} hjalp {pronoun_object} med å jobbe på bilen.		possessiv
+Kan du overlevere {possessive} bok til {pronoun_object}?		possessiv
+Mora til {pronoun_object} er mer stolt av {possessive} prestasjon.	{pronoun_subject} si mor er mer stolt av {possessive} prestasjon.	possessiv
+{'pronoun_subject} gråt da {pronoun_subject} sa at innsatsen {possessive} ikke var nok.	{'pronoun_subject} gråt da {pronoun_subject} sa at {possessive} sin innsats ikke var nok.	possessiv
diff --git a/locale/no/translations.suml b/locale/no/translations.suml
index eb35af9f3..f0c45585c 100644
--- a/locale/no/translations.suml
+++ b/locale/no/translations.suml
@@ -630,7 +630,6 @@ faq:
                     Før du rapporterer en feil, kan du prøve å slå av slike utvidelser for å se
                     om problemet forsvinner.
 
-
 links:
     header: 'Referanser'
     headerLong: 'Ekstra referanser'
@@ -748,7 +747,6 @@ contact:
                 Mange seksjoner på nettsiden (substantiv, inklusiv ordbok...) har alternativer for å legge til og
                 redigere eksisterende bidrag. Hvis du tror at noe mangler eller er feil, nøl ikke med å ta disse i bruk.
 
-
 workshops:
     header: 'Opplæring'
     headerLong: 'Opplæring'
@@ -1003,12 +1001,6 @@ profile:
         close: 'Bare hvis vi er nærme'
         meh: 'Tja'
         no: 'Nei'
-    banner: >
-        Du kan også bruke nettsiden vår til å lage et kort, {/@example=sånn som denne},
-        som inneholder navnene dine, pronomen, pride flagg, likte ord, osv.
-        Så kan du lenke den til din bio eller epost-signatur.
-        Bare lag en konto {/konto=her}.
-    bannerButton: 'Lag et kort'
     card:
         link: 'Kortbilde'
         generate: 'Generer'
@@ -1475,7 +1467,6 @@ translationMode:
     changes: 'Endringer'
     commit: 'Lagre endringer'
     revert: 'Angre endringer'
-    action: 'Foreslå oversettelser'
     pause: 'Sett oversettelsesmodus på pause'
 
 flags:
diff --git a/server/express/pronouns.ts b/server/express/pronouns.ts
index d79a80d82..ff06f99f7 100644
--- a/server/express/pronouns.ts
+++ b/server/express/pronouns.ts
@@ -27,8 +27,8 @@ const pronounLibrary = new PronounLibrary(global.config, pronounGroups, pronouns
 const buildExample = (e: PronounExamplesData): Example => new Example(
     Example.parse(e.singular),
     Example.parse(e.plural || e.singular),
-    Example.parse(e['null'] || e.singular),
-    Example.parse(e['null_plural'] || e.plural || e.singular),
+    Example.parse(e.null || e.singular),
+    Example.parse(e.null_plural || e.plural || e.singular),
     e.isHonorific,
 );
 
diff --git a/src/classes.ts b/src/classes.ts
index 655984cc4..c9ba4ff22 100644
--- a/src/classes.ts
+++ b/src/classes.ts
@@ -22,13 +22,15 @@ export class Example {
     singularNullParts: ExamplePart[];
     pluralNullParts: ExamplePart[];
     isHonorific: boolean;
+    categories: string[];
 
-    constructor(singularParts: ExamplePart[], pluralParts: ExamplePart[], singularNullParts: ExamplePart[], pluralNullParts: ExamplePart[], isHonorific = false) {
+    constructor(singularParts: ExamplePart[], pluralParts: ExamplePart[], singularNullParts: ExamplePart[], pluralNullParts: ExamplePart[], isHonorific = false, categories: string[] = []) {
         this.singularParts = singularParts;
         this.pluralParts = pluralParts;
         this.singularNullParts = singularNullParts;
         this.pluralNullParts = pluralNullParts;
         this.isHonorific = isHonorific;
+        this.categories = categories;
     }
 
     static parse(str: string): ExamplePart[] {
@@ -144,7 +146,8 @@ export class ExampleCategory {
         }
         return config.pronouns.exampleCategories.map((exampleCategory) => {
             const matchingExamples = examples.filter((example) => {
-                return exampleCategory.morphemes.some((morpheme) => example.hasMorpheme(morpheme));
+                return exampleCategory.morphemes?.some((morpheme) => example.hasMorpheme(morpheme)) ||
+                    example.categories?.includes(exampleCategory.name);
             });
             return new ExampleCategory(exampleCategory.name, matchingExamples, exampleCategory.comprehensive);
         });
diff --git a/src/data.ts b/src/data.ts
index d86c94914..8834540c9 100644
--- a/src/data.ts
+++ b/src/data.ts
@@ -17,9 +17,10 @@ export const examples = buildList(function* () {
         yield new Example(
             Example.parse(e.singular),
             Example.parse(e.plural || e.singular),
-            Example.parse(e['null'] || e.singular),
-            Example.parse(e['null_plural'] || e.plural || e.singular),
+            Example.parse(e.null || e.singular),
+            Example.parse(e.null_plural || e.plural || e.singular),
             e.isHonorific,
+            e.categories ? e.categories.split(',').map(c => c.trim()).filter(c => c) : [],
         );
     }
 });