Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/list topic comment #205

Closed
wants to merge 35 commits into from
Closed

Feat/list topic comment #205

wants to merge 35 commits into from

Conversation

edelagnier
Copy link
Contributor

Closes #113
Closes #200

Proposition de fonctionnalité

Utilisateur,
Quand je me rends sur une page bouquet, je peux consulter la liste des discussions liées à ce bouquet

Éléments supplémentaires de ma proposition

Ayant été au plus cours, je n'ai pas rajouté de test sur le code existant, @bonjourmauko aurait le temps de rajouter un mock ?

La fonctionnalité n'est que peu testable car les bouquets n'ont actuellement pas de commentaires. Mais voici un exemple de rendu en chargeant artificiellement les commentaires d'un dataset à la place :
image

Verified

This commit was signed with the committer’s verified signature.
targos Michaël Zasso

Verified

This commit was signed with the committer’s verified signature. The key has expired.
danielleadams Danielle Adams

Verified

This commit was signed with the committer’s verified signature.
targos Michaël Zasso
Copy link

netlify bot commented Dec 1, 2023

Deploy Preview for ecospheres ready!

Name Link
🔨 Latest commit 01fb9cd
🔍 Latest deploy log https://app.netlify.com/sites/ecospheres/deploys/657364f2589bf800083c2c5d
😎 Deploy Preview https://deploy-preview-205--ecospheres.netlify.app
📱 Preview on mobile
Toggle QR Code...

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
Copy link
Contributor

streino commented Dec 1, 2023

J'ai un 500 lorsque j'affiche un bouquet sans discussions :
https://demo.data.gouv.fr/api/1/discussions/?for=undefined&page=1

@edelagnier
Copy link
Contributor Author

@streino bien vu, c'est corrigé

Verified

This commit was signed with the committer’s verified signature.
targos Michaël Zasso
@edelagnier edelagnier force-pushed the feat/list_topic_comment branch from c5f2912 to 3061f9b Compare December 1, 2023 15:12
@bonjourmauko bonjourmauko added this to the V1 : Découvrabilité par cas d’usage milestone Dec 4, 2023
@bonjourmauko bonjourmauko added the enhancement Améliorations et nouvelles fonctionnalités label Dec 4, 2023
@bonjourmauko
Copy link
Contributor

@streino Il n'y a pour le moment que des conflits au niveau du CSS et des types, faciles à règler. Je fais une passe maintenant sur les autres PR pour voir s'il y a d'autres conflits potentiels.

Une fois que #210 sera intégrée ici on te demandera la review, pas besoin de t'embêter avant.

@bonjourmauko
Copy link
Contributor

Les changements sur #211 s'adressent à la même base du code (DatasetDetailView) mais (miraculeusement ?) ne rentrent pas directement en conflit avec cette PR (au contraire ça donne une belle opportunité de refactoring a posteriori).

fix: broken comments
src/store/DiscussionStore.ts Outdated Show resolved Hide resolved
errorValue.value = null
}

return {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est nécessaire d'exporter les func internes ? A l'utilisation, on a que :

const {
  getDiscussions,
  discussionsPage,
  discussionsPages,
  error: discussionsError,
  loading: discussionsLoading
} = storeToRefs(discussionStore)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Faudrait tester ! La doc est un peu confuse :

Note you must return all state properties in setup stores for pinia to pick them up as state. In other words, you cannot have private state properties in stores. Not returning all state properties can break SSR, devtools, and other plugins.

https://pinia.vuejs.org/core-concepts/

src/views/datasets/DatasetDetailView.vue Outdated Show resolved Hide resolved
@bonjourmauko bonjourmauko requested a review from streino December 8, 2023 18:29
@streino
Copy link
Contributor

streino commented Dec 12, 2023

Il va falloir articuler ça avec #263.
Je passe le témoin à @abulte qui sera beaucoup plus pertinent à ce sujet.

@streino streino requested a review from abulte December 12, 2023 17:22
@streino streino assigned streino and abulte and unassigned streino Dec 14, 2023
@streino
Copy link
Contributor

streino commented Dec 14, 2023

@abulte a initié l'articulation en #283.
La présente PR est donc obsolète, @abulte je te laisse la fermer quand tu n'en as plus besoin.

@abulte abulte closed this Jan 11, 2024
@abulte abulte deleted the feat/list_topic_comment branch October 28, 2024 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Améliorations et nouvelles fonctionnalités
Projects
None yet
Development

Successfully merging this pull request may close these issues.

api(social): lister commentaires d'un bouquet feat(social): lister commentaires d'un bouquet
4 participants