Trie les articles de workspaces par date de mise à jour #1173
Conversation
On garantit un fonctionnement similaire à d'autres endroits de l'application, avec un code plus facile à lire.
✅ Deploy Preview for stylo-docs canceled.
|
| return Article.getArticles({ | ||
| filter: { _id: { $in: workspace.articles } }, | ||
| loaders: context.loaders, | ||
| }) |
There was a problem hiding this comment.
C'est malin je n'y avais pas pensé 👍🏻
Est-ce qu'il y a une différence notable dans les performances ? Si tu associes plusieurs centaines/milliers d'articles à un espace de travail ?
There was a problem hiding this comment.
Je ne sais pas, c'est encore mystérieux le fonctionnement des loaders (ça appelle .complete() quand même, donc ça ne change pas grand chose à la précédente manière de faire ?)
There was a problem hiding this comment.
De prime abord, je dirais que ta solution est plus rapide.
Comme le code est plus simple et que je pense qu'on a assez peu d'espace de travail avec plus de 50 articles on s'en fiche un peu. C'était plus par curiosité 😅
There was a problem hiding this comment.
Pour les data loaders ça permet de limiter l'effet des requêtes N+1
https://www.apollographql.com/tutorials/dataloaders-dgs/03-data-loaders-under-the-hood
thom4parisot
had a problem deploying
to
github-pages
GitHub Actions
Failure
Comme dans les espaces personnels.
fixes #1172