feat(social): commente un bouquet #180

bonjourmauko · 2023-11-23T14:02:51Z

Closes #128
Depends on #198
Depends on #205

Changement technique

Contexte ou situation

Je m'attends à pouvoir créer un commentaire sur un bouquet.

Problème rencontré par les utilisateurs

Ce n'est pas le cas, mais c'est possible via l'API data.gouv.fr.

Proposition de solution au problème

Implémente la création d'une discussion autour d'un bouquet.

Qu'est-ce qui change ?

Modifications techniques

🤟 Commenter un bouquet (via l'API).

netlify · 2023-11-23T14:02:56Z

✅ Deploy Preview for ecospheres ready!

Name	Link
🔨 Latest commit
🔍 Latest deploy log	https://app.netlify.com/sites/ecospheres/deploys/655f83477dfa151ac49da981
😎 Deploy Preview	https://deploy-preview-180--ecospheres.netlify.app/
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

streino · 2023-11-23T17:15:38Z

Pourquoi ne pas commencer par la visualisation des discussions, pour pouvoir ensuite implémenter la création de commentaires de bout en bout ?

Nitpick : Ce serait bien de séparer les cleanups dans des PRs séparées pour faciliter la review. Les PR peuvent être stacked (EC-128-Discussion-bouquet -> cleanup -> main), comme ça EC-128-Discussion-bouquet est automatiquement rebased sur main lorsque cleanup est mergée.

bonjourmauko · 2023-11-24T05:25:01Z

Pourquoi ne pas commencer par la visualisation des discussions, pour pouvoir ensuite implémenter la création de commentaires de bout en bout ?

Cette functionnalité à effectivement plusieurs entrées :

si je suis en train de créer un bouquet : commenter -> visualiser
si je suis en train de naviguer les bouquets : visualiser -> commenter

J'ai proposé lors de l'atelier de démarrage qu'on commence comme tu dis :

d'abord la navigation des bouquets (avec des bouquets fake)
et ensuite la création des bouquets

Il a été acté de commencer par la création, et ce ticket suit tout simplement cette logique.

Nitpick : Ce serait bien de séparer les cleanups dans des PRs séparées pour faciliter la review. Les PR peuvent être stacked (EC-128-Discussion-bouquet -> cleanup -> main), comme ça EC-128-Discussion-bouquet est automatiquement rebased sur main lorsque cleanup est mergée.

Effectivement ! Bonne idée.

(Note tout de même que le format est automatique maintenant à chaque commit)

streino · 2023-11-24T10:01:13Z

Cette functionnalité à effectivement plusieurs entrées :
* si je suis en train de créer un bouquet : commenter -> visualiser
* si je suis en train de naviguer les bouquets : visualiser -> commenter

J'ai du mal à conceptualiser le 1er cas.
Pour moi dans tous les cas on a visualiser -> commenter.
Même lorsque la discussion n'existe pas encore il faut la visualiser, par exemple data.gouv :

Il a été acté de commencer par la création, et ce ticket suit tout simplement cette logique.

Pour la création de bouquets, oui. Mais je ne vois pas en quoi la décision sur les bouquets s'applique automatiquement sur les discussions. Selon la même logique, on a une section qui visualise les jeux de données dans la création de bouquets, mais on a pas décidé de commencer par la création de jeux de données.

(Note tout de même que le format est automatique maintenant à chaque commit)

Comment se fait-il qu'on ait encore des reformat automatiques si le format est fait à chaque commit ?

Je regarderai si on peut appliquer le format qu'aux fichiers modifiés qui sont commit, pour éviter de polluer automatiquement les PR. Sinon c'est vrai que ça demande un peu de gitfu pour récupérer le truc. En attendant, on peut aussi tjs passer manuellement le format avant.

bonjourmauko · 2023-11-24T14:41:15Z

Cette functionnalité à effectivement plusieurs entrées :

si je suis en train de créer un bouquet : commenter -> visualiser

si je suis en train de naviguer les bouquets : visualiser -> commenter

J'ai du mal à conceptualiser le 1er cas. Pour moi dans tous les cas on a visualiser -> commenter. Même lorsque la discussion n'existe pas encore il faut la visualiser, par exemple data.gouv :

Il a été acté de commencer par la création, et ce ticket suit tout simplement cette logique.

Pour la création de bouquets, oui. Mais je ne vois pas en quoi la décision sur les bouquets s'applique automatiquement sur les discussions. Selon la même logique, on a une section qui visualise les jeux de données dans la création de bouquets, mais on a pas décidé de commencer par la création de jeux de données.

J'ai déjà dit qu'il me semble plus cohérent de commencer par la visualisation, donc je n'ai pas d'autre élément à apporter. En revanche, je vous laisse voir avec @edelagnier ce qui vous semble plus pratique côté implémentation.

(Note tout de même que le format est automatique maintenant à chaque commit)

Comment se fait-il qu'on ait encore des reformat automatiques si le format est fait à chaque commit ?

Je regarderai si on peut appliquer le format qu'aux fichiers modifiés qui sont commit, pour éviter de polluer automatiquement les PR. Sinon c'est vrai que ça demande un peu de gitfu pour récupérer le truc. En attendant, on peut aussi tjs passer manuellement le format avant.

J'avoue. Une autre idée, ce serait de faire #157 et #65, si tu te sens motivé 🙏 😃 (cc @abulte aussi)

bonjourmauko · 2023-11-28T17:20:00Z

⚠️ rebased and force-pushed cc @edelagnier

bonjourmauko · 2023-11-28T17:42:36Z

@streino Pour faciliter la discussion, j'ai réduit le périmètre de cette PR à #199 (API) et te propose qu'on poursuive la discussion concernant le côté front ici. Si cela te convient 😃

edelagnier · 2023-11-29T09:11:59Z

bon pour moi cote code d'appel API, préférez vous qu'on attende d'avoir l'interface graphique pour merger ?

streino

Ok pour moi aussi (modulo ma question sur le await), mais j'ai tendance à préférer une PR bout-en-bout sur des petites modifs ; ça clarifie l'usage et permet de tester la fonctionnalité directement.
Vu que la PR est prête, faites comme vous le sentez.

streino · 2023-11-29T09:11:15Z

src/services/api/DatagouvfrAPI.ts

-  async delete(entityId) {
-    return this.httpClient.delete(`${this.url()}/${entityId}/`).then(
+  async delete(entityId: string): Promise<Response> {
+    return await this.httpClient.delete(`${this.url()}/${entityId}/`).then(


Le await est nécessaire ?

@edelagnier ? (Il me semble que c'est le npm run format qui rajoute ça automatiquement).

bonjourmauko · 2023-11-29T15:09:36Z

src/services/api/DatagouvfrAPI.ts

-  #handleError({ response, message }) {
-    if (response) return { status: response.status }
-    return { message }
+  #handleError({ response, message }): Response {


@edelagnier vu que c'est du TypeScript maintenant, ne pourrait-on utiliser private plutôt ?

Suggested change

#handleError({ response, message }): Response {

private handleError({ response, message }): Response {

bonjourmauko · 2023-12-06T12:40:25Z

J'ai marqué cette PR comme draft, car elle depend fortement de #205.

bonjourmauko added the enhancement Améliorations et nouvelles fonctionnalités label Nov 23, 2023

bonjourmauko added this to the V1 : Découvrabilité par cas d’usage milestone Nov 23, 2023

bonjourmauko requested review from abulte and edelagnier November 23, 2023 14:02

bonjourmauko assigned YeLnatSs Nov 23, 2023

bonjourmauko requested a review from streino November 23, 2023 14:04

streino assigned edelagnier and unassigned YeLnatSs Nov 24, 2023

bonjourmauko mentioned this pull request Nov 28, 2023

Divergence entre npm run format et npm run lint #191

Open

1 task

bonjourmauko changed the title ~~feat(api): commente un bouquet~~ api(social): commente un bouquet Nov 28, 2023

bonjourmauko added api and removed enhancement Améliorations et nouvelles fonctionnalités labels Nov 28, 2023

bonjourmauko changed the base branch from main to build/applique-format November 28, 2023 17:14

Marvin SAURAYE added 2 commits November 28, 2023 18:17

Add test on discussion API

de154b3

Add check for status

212dd99

bonjourmauko force-pushed the EC-128-Discussion-bouquet branch from e6c65f8 to 773eebe Compare November 28, 2023 17:17

refactor(test): proper error handling (#128)

4b68205

bonjourmauko force-pushed the EC-128-Discussion-bouquet branch from 773eebe to 4b68205 Compare November 28, 2023 17:19

bonjourmauko mentioned this pull request Nov 28, 2023

feat(social): commenter un bouquet #128

Closed

4 tasks

Base automatically changed from build/applique-format to main November 29, 2023 00:12

streino approved these changes Nov 29, 2023

View reviewed changes

bonjourmauko commented Nov 29, 2023

View reviewed changes

bonjourmauko changed the base branch from main to feat/list_topic_comment December 4, 2023 14:42

bonjourmauko marked this pull request as draft December 6, 2023 12:39

bonjourmauko unassigned edelagnier Dec 6, 2023

bonjourmauko changed the title ~~api(social): commente un bouquet~~ feat(social): commente un bouquet Dec 6, 2023

bonjourmauko added enhancement Améliorations et nouvelles fonctionnalités and removed api labels Dec 6, 2023

abulte closed this Jan 11, 2024

abulte deleted the EC-128-Discussion-bouquet branch October 28, 2024 16:23

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(social): commente un bouquet #180

feat(social): commente un bouquet #180

bonjourmauko commented Nov 23, 2023 •

edited

Loading

netlify bot commented Nov 23, 2023 •

edited

Loading

streino commented Nov 23, 2023

bonjourmauko commented Nov 24, 2023

streino commented Nov 24, 2023

bonjourmauko commented Nov 24, 2023

bonjourmauko commented Nov 28, 2023

bonjourmauko commented Nov 28, 2023

edelagnier commented Nov 29, 2023

streino left a comment

streino Nov 29, 2023

bonjourmauko Nov 29, 2023

bonjourmauko Nov 29, 2023

bonjourmauko commented Dec 6, 2023

	#handleError({ response, message }): Response {
	private handleError({ response, message }): Response {

feat(social): commente un bouquet #180

feat(social): commente un bouquet #180

Conversation

bonjourmauko commented Nov 23, 2023 • edited Loading

Changement technique

Contexte ou situation

Problème rencontré par les utilisateurs

Proposition de solution au problème

Qu'est-ce qui change ?

Modifications techniques

netlify bot commented Nov 23, 2023 • edited Loading

✅ Deploy Preview for ecospheres ready!

streino commented Nov 23, 2023

bonjourmauko commented Nov 24, 2023

streino commented Nov 24, 2023

bonjourmauko commented Nov 24, 2023

bonjourmauko commented Nov 28, 2023

bonjourmauko commented Nov 28, 2023

edelagnier commented Nov 29, 2023

streino left a comment

Choose a reason for hiding this comment

streino Nov 29, 2023

Choose a reason for hiding this comment

bonjourmauko Nov 29, 2023

Choose a reason for hiding this comment

bonjourmauko Nov 29, 2023

Choose a reason for hiding this comment

bonjourmauko commented Dec 6, 2023

bonjourmauko commented Nov 23, 2023 •

edited

Loading

netlify bot commented Nov 23, 2023 •

edited

Loading