fix: correction et ajout de certains types manquants #1464
Conversation
|
| @@ -0,0 +1,74 @@ | |||
| import { Theme, ThemeContentRelation } from "../types/themes"; | |||
There was a problem hiding this comment.
J'ai migré le fichier buildTheme en typescript.
| @@ -18,7 +18,8 @@ const graphQLThemesQuery = ` | |||
| slug | |||
| source | |||
| title | |||
| document | |||
There was a problem hiding this comment.
J'ai modifié la requête hasura pour fetch les thèmes car elle récupérait les documents complets alors que l'on utilise que la description et l'url. Ca fait un json beaucoup plus léger !
| @@ -6,7 +8,7 @@ export interface Theme { | |||
| cdtnId: string; | |||
| id: string; | |||
| slug: string; | |||
| source: string; | |||
| source: "themes"; | |||
There was a problem hiding this comment.
On est dans un thème, la source ne peut être que themes.
| type: "external", | ||
| })) | ||
| .map( | ||
| (item): MailTemplateReference => ({ |
There was a problem hiding this comment.
Ici j'ai du ajouter le type de retour car Typescript n'arrivait pas à matcher le typeavec external 🤷
m-maillot
temporarily deployed
to
review-auto
GitHub Actions
Inactive
There was a problem hiding this comment.
Dommage qu'on ait pas pu repondre les types dans le package partagé
There was a problem hiding this comment.
Ce sont des types lié à la BDD. Actuellement, ils ne sont pas dans le shared. Il faudrait les ajouter en effet pour avoir chaque layer de dispo
|
🎉 Deployment for commit e3fa73b : Docker images
|
|
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation. |
No description provided.