-
Notifications
You must be signed in to change notification settings - Fork 7
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
Ajoute une fonction pour récupérer les tags catégories #3445
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
) {} | ||
|
||
router = this.trpc.router({ | ||
getCategoriesByCollectivite: this.trpc.authedProcedure |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Vu que tu es déjà dans un router dédié au get des catégories, j'aurais tendance à simplifier le naming avec un simple list
par exemple. Pour avoir une lisibilité de l'appel un poil plus concis trpc.tags.categories.list
.
(en omettant le byCollectivite
qui est tellement systématique pour la majorité de nos appels qu'il devient juste bruyant de l'expliciter IMO).
Détails de naming, tu me diras ce que tu en penses.
imports: [ | ||
ConfigurationModule, | ||
CommonModule, | ||
TrpcModule, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Micro détail / partage : tu peux te passer de l'import de TrpcModule
que j'ai identifié en tant que @Global()
, et donc accessible par toues les autres modules sans besoin d'importer.
b89c698
to
9778d86
Compare
…egorie_tag disponibles pour une collectivité.
9778d86
to
3873ca3
Compare
Appel trpc :