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

Suppression de certains appels à .lean() #1028

Closed
wants to merge 1 commit into from
Closed

Conversation

thom4parisot
Copy link
Member

Ça venait masquer des contenus où certaines infos étaient devenues obligatoires entre temps (les couleurs de tags).

refs #1027

Copy link

netlify bot commented Oct 1, 2024

Deploy Preview for stylo-docs canceled.

Name Link
🔨 Latest commit 3562131
🔍 Latest deploy log https://app.netlify.com/sites/stylo-docs/deploys/66fbbd3e6a3e2e0008dcd4fc

@ggrossetie
Copy link
Collaborator

De mémoire je crois qu'on avait fait ça pour des raisons de performance sur l'écran qui liste les articles. Peut être qu'il faudrait voir les usages afin d'utiliser lean ou non en fonction du contexte (i.e., chargement unitaire ou chargement de plusieurs éléments)

@thom4parisot
Copy link
Member Author

thom4parisot commented Oct 14, 2024

Problème résolu en lançant ces requêtes Mongo :

db.tags.updateMany({ color: '' }, { $set: { color: '#ccc' } })
db.tags.updateMany({ color: 'grey' }, { $set: { color: '#ccc' } })

Est-ce le but de 6dc95a4 ? Et peut-être que c'était l'intention de ce script que de résoudre ce problème ?

@ggrossetie
Copy link
Collaborator

Est-ce le but de 6dc95a4 ? Et peut-être que c'était l'intention de ce script que de résoudre ce problème ?

Oui exactement 👍
On cloture ?

@thom4parisot thom4parisot deleted the fix/color-type branch October 15, 2024 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants