-
Notifications
You must be signed in to change notification settings - Fork 6
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: Ajoute la date de mise à jour aux carte d'actu #3509
base: main
Are you sure you want to change the base?
Conversation
f78b652
to
52e6ff7
Compare
52e6ff7
to
a3b8e90
Compare
a3b8e90
to
6f94795
Compare
J'ai joué un peu avec typescript, on peut faire un type générique pour représenter les props sérialisées : type Serialized<BaseType> = {
[Key in keyof BaseType]:
BaseType[Key] extends boolean ? boolean
: BaseType[Key] extends number ? number
: BaseType[Key] extends string ? string
: BaseType[Key] extends Record<string, unknown> ? Serialized<BaseType[Key]>
: BaseType[Key] extends Iterable<unknown> ? Serialized<BaseType[Key]>
: BaseType[Key] extends { toString: () => string } ? string
: never
} Edit : V2 qui inclu les primitives : type Serialized<BaseType> =
BaseType extends boolean ? boolean
: BaseType extends number ? number
: BaseType extends string ? string
: BaseType extends Record<string, unknown> | Iterable<unknown> ? {
[Key in keyof BaseType]: Serialized<BaseType[Key]>
}
: BaseType extends { toString: () => string } ? string
: never |
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.
Je suis plutôt aligné avec les changements. J'ai pas bien analysé le générique que tu as montré mais je pense que le sujet des dates va revenir et on pourra le traiter à ce moment-là.
Je trouve que les index
peuvent se gonfler pas mal entre l'appel des props, la page et maintenant la sérialisation ce qui est flagrant sur l'accueil. Mais pareil je pense pas qu'on soit obligé de le traiter dans cette PR on peut se faire une tache tech pour essayer de simplifier la page d'accueil à la rentrée (surtout qu'on sait qu'il y a peut-être des plans pour une refonte ici)
No description provided.