migration: postgres mes-adresses-api #945
Conversation
|
BUG BUILD GITHUB: vercel/next.js#69263 |
fufeck
changed the title
contexts/markers.tsx
Outdated
| return [...prevMarkers, { _id: uniqueId(), ...marker }]; | ||
| return [ | ||
| ...prevMarkers, | ||
| { id: new Types.ObjectId().toHexString(), ...marker }, |
There was a problem hiding this comment.
Pour les positions je trouve que ça serait plus élégant d'utiliser des UUIDs, là côté front ça nous fait importer mongoose juste pour générer les objectIds et côté back on utilise la fonction custom pour la primary key alors qu'un petit @PrimaryGeneratedColumn("uuid") ferait le taff
There was a problem hiding this comment.
J'ai fait un petit analyze, mongoose prend une place monstre... Si tu veux pas pour les uuid, je dirais on continue avec le uniqueId de lodash côté front et on détecte côté back si c'est une insertion on génère un ObjectId (sur le format de l'ID)
There was a problem hiding this comment.
J'ai opté pour la deuxième solution, comme il y a this.id = new ObjectId().toHexString();, si l'on remet la ligne return [...prevMarkers, { _id: uniqueId(), ...marker }]; cela marche quand même (l'id est écrasé par le BeforeInsert de l'entity position) donc pas de soucis ;)
PR