3042 add field in user #3067

cspriet · 2024-12-27T16:37:52Z

@alice-telescoop pas sûre de la migration !

alice-telescoop · 2024-12-30T10:57:09Z

packages/api/migrations/20241227-add-user-from.js

Ta migration marche :)
Dans l'esprit il faudrait down qui enlève les champs mais on les utilise presque jamais.
Et la requête est pas écrite de la façon la + simple qui me vient en tête donc je t'écris ce que j'aurais fait. Je ne sais pas dire s'il ce serait mieux en terme de perfs

alice-telescoop · 2024-12-30T10:58:29Z

packages/api/migrations/20241227-add-user-from.js

+        await collection
+            .aggregate([
+                {
+                    $addFields: {
+                        from: [],
+                        fromEmail: "",
+                        fromOther: "",
+                    },
+                },
+                { $out: "users" },
+            ])
+            .toArray();


Suggested change

await collection

.aggregate([

{

$addFields: {

from: [],

fromEmail: "",

fromOther: "",

},

},

{ $out: "users" },

])

.toArray();

await collection

.updateMany({}, {

$set: {

from: [],

fromEmail: "",

fromOther: "",

}

})

Les aggregate sont souvent nécessaires pour des choses plus complexes que ça

je partage la vision d'Alice

@alice-telescoop @mxmeunier et concernant le nom du fichier ?
J'ai mis la date du jour mais le reste j'ai pas réussi à déterminer d'où cela vient.

alice-telescoop · 2024-12-30T11:01:02Z

packages/dto/auth/subscriptionForm/fromType.ts

Les intitulés ne me parlent pas trop. Je te mets des propositions mais c'est subjectif. La vraie chose à faire c'est de mettre des commentaires pour clarifier, que tu changes ou non

alice-telescoop · 2024-12-30T11:01:45Z

packages/dto/auth/subscriptionForm/fromType.ts

@@ -0,0 +1,7 @@
+export enum FromTypeEnum {
+    INTRODUCTION = "INTRODUCTION",


Suggested change

INTRODUCTION = "INTRODUCTION",

DEMO = "DEMO",

alice-telescoop · 2024-12-30T11:01:59Z

packages/dto/auth/subscriptionForm/fromType.ts

+    INTRODUCTION = "INTRODUCTION",
+    RESEARCH = "RESEARCH",
+    COLLEAGUES_HIERARCHY = "COLLEAGUES_HIERARCHY",
+    POST = "POST",


Suggested change

POST = "POST",

SOCIALS = "SOCIALS",

alice-telescoop · 2024-12-30T11:03:01Z

packages/dto/auth/subscriptionForm/fromType.ts

@@ -0,0 +1,7 @@
+export enum FromTypeEnum {
+    INTRODUCTION = "INTRODUCTION",
+    RESEARCH = "RESEARCH",


Suggested change

RESEARCH = "RESEARCH",

OWN_SEARCH = "OWN_SEARCH",

Celui-là je suis pas convaincue mais RESEARCH ça m'évoque la recherche scientifique

Ca veut pas dire grand chose je trouve. On parle de quoi, une recherche personnelle sur le navigateur comme "données sur les subventions des associations" avec google ?

C'est ce que je comprends mais il faudrait voir avec Marion pour être sûrs

Oui effectivement je suis d'accord @alice-telescoop avec tes idées. J'ai essayé de retranscrire les intitulés des champs mais effectivement pas super parlant.
Pour recherche, dans le front l'intitulé c'est "Via des recherches sur Internet", c'est plutôt moi qui ait mal traduit je pense, est-ce que "SEARCH_ENGINE" ça vous paraît plus cohérent ? @mxmeunier

alice-telescoop · 2024-12-30T11:03:29Z

J'ai cliqué trop vite sur comment et pas approve mais ça me va comme ça

mxmeunier · 2025-01-07T10:09:09Z

packages/api/migrations/20241227-add-user-from.js

+        await collection
+            .aggregate([
+                {
+                    $addFields: {
+                        from: [],
+                        fromEmail: "",
+                        fromOther: "",
+                    },
+                },
+                { $out: "users" },
+            ])
+            .toArray();


je partage la vision d'Alice

mxmeunier · 2025-01-07T10:11:21Z

packages/api/src/modules/user/services/profile/user.profile.service.test.ts

+                });
+                expect(actual).toMatchSnapshot();
+            });
+            it("should return true", () => {


tu passes pas vraiment de "from" ? C'est biaisé comme test, non ?

Ah oui effectivement... Mieux maintenant ?

mxmeunier · 2025-01-07T10:11:50Z

packages/api/src/modules/user/services/profile/user.profile.service.test.ts

@@ -156,6 +156,26 @@ describe("user profile service", () => {
            });
            // TODO question : laisser le test comme ça ou extraire des tests ?
        });
+
+        describe("from", () => {


je comprend pas trop le nom du describe. On test plus bas validateUserProfileData

@mxmeunier c'est parce que j'ai nommé le champ en base "from". Est-ce que c'est mieux ?

mxmeunier · 2025-01-07T10:15:10Z

packages/dto/auth/subscriptionForm/fromType.ts

@@ -0,0 +1,7 @@
+export enum FromTypeEnum {
+    INTRODUCTION = "INTRODUCTION",
+    RESEARCH = "RESEARCH",


Ca veut pas dire grand chose je trouve. On parle de quoi, une recherche personnelle sur le navigateur comme "données sur les subventions des associations" avec google ?

mxmeunier · 2025-01-07T10:17:16Z

packages/dto/auth/subscriptionForm/fromType.ts

+    INTRODUCTION = "INTRODUCTION",
+    RESEARCH = "RESEARCH",
+    COLLEAGUES_HIERARCHY = "COLLEAGUES_HIERARCHY",
+    POST = "POST",


Ca a sûrement été travaillé avec les bizdev, ils entendent quoi derrière chaque mot ? Car POST ça peut être linkedin mais aussi beta.gouv ou n'importe quoi d'autre non ? Il me semblait que le besoin était précis donc pourquoi pas avoir carrément une granularité sur "LINKEDIN" / "NEWSLETTER" / "Facebook" / etc

Tu peux trouver une référence sur la maquette de Marion dans le ticket. Ensuite les mots de l'enum c'est nous qui voyons mais je suis d'accord que je les trouve pas très clairs

…-asso into 3042-add-field-in-user

cspriet added 2 commits December 27, 2024 16:59

feat(api,dto): add from fields in user

a7e06dc

feat(api,dto): add migration

76cbea2

cspriet requested a review from alice-telescoop December 27, 2024 16:37

cspriet linked an issue Dec 27, 2024 that may be closed by this pull request

Ajouter un champ dans l'objet utilisateur pour la provenance #3042

Open

alice-telescoop reviewed Dec 30, 2024

View reviewed changes

mxmeunier reviewed Jan 7, 2025

View reviewed changes

cspriet added 5 commits January 8, 2025 14:16

feat(api,dto): alice & maxime comments

b2d48f4

Merge branch 'develop' of https://github.com/betagouv/api-subventions…

37470ca

…-asso into 3042-add-field-in-user

feat(api): snapshot

fd43281

feat(api): snapshot

ec78935

feat(api): snapshot

f854539

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

3042 add field in user #3067

3042 add field in user #3067

cspriet commented Dec 27, 2024 •

edited by alice-telescoop

Loading

alice-telescoop Dec 30, 2024

alice-telescoop Dec 30, 2024

mxmeunier Jan 7, 2025

cspriet Jan 8, 2025

alice-telescoop Dec 30, 2024

alice-telescoop Dec 30, 2024

alice-telescoop Dec 30, 2024

alice-telescoop Dec 30, 2024

mxmeunier Jan 7, 2025

alice-telescoop Jan 7, 2025

cspriet Jan 8, 2025

alice-telescoop commented Dec 30, 2024

mxmeunier Jan 7, 2025

mxmeunier Jan 7, 2025

cspriet Jan 8, 2025

mxmeunier Jan 7, 2025

cspriet Jan 8, 2025

mxmeunier Jan 7, 2025

mxmeunier Jan 7, 2025

alice-telescoop Jan 7, 2025

		@@ -0,0 +1,7 @@
		export enum FromTypeEnum {
		INTRODUCTION = "INTRODUCTION",

3042 add field in user #3067

Are you sure you want to change the base?

3042 add field in user #3067

Conversation

cspriet commented Dec 27, 2024 • edited by alice-telescoop Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

alice-telescoop commented Dec 30, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

cspriet commented Dec 27, 2024 •

edited by alice-telescoop

Loading