migration: postgres #470
migration: postgres #470
Conversation
| @@ -64,7 +66,7 @@ export class SearchQueryPipe implements PipeTransform { | |||
|
|
|||
| if (query.email) { | |||
| if (typeof query.email === 'string' && checkValidEmail(query.email)) { | |||
| filters.emails = { $regex: new RegExp(`^${query.email}$`, 'i') }; | |||
| filters.emails = ArrayContains([query.email]); | |||
There was a problem hiding this comment.
C'est bien case insensitive?
apps/cron/src/cron.module.ts
Outdated
| keepConnectionAlive: true, | ||
| schema: 'public', | ||
| migrationsRun: true, | ||
| migrations: [Initialization1724762471013], |
There was a problem hiding this comment.
On a besoin de run la migration d'initialisation dans le CRON si elle est déjà faite par l'API? Je pense que ça serait mieux de laisser toutes les migrations côté API
There was a problem hiding this comment.
Négatif, c'est l'appli qui démare en première qui init la DB
| @@ -53,36 +44,11 @@ export class CronService { | |||
|
|
|||
| @Cron(CronExpression.EVERY_DAY_AT_2AM) | |||
| async removeSoftDeletedBALsOlderThanOneYear() { | |||
| console.debug('Task start : purge old deleted BALs'); | |||
There was a problem hiding this comment.
On laisse pas les logs? ça peut toujours être util
There was a problem hiding this comment.
Pas besoin, normalement il y a deja de console.log dans TaskQueue
| @@ -74,12 +80,15 @@ | |||
| "mongodb": "^6.1.0", | |||
There was a problem hiding this comment.
On peut virer mongodb non? et le plugin mongoose lean?
There was a problem hiding this comment.
Je vais dégager mongoose plutot dont on a pas besoin, mongodb obligatoire pour ObjectId
|
TESTS :
Les positions des toponymes ont été importés mais pas celles des numéros. Par contre après quand j'en crée dans la BAL, pas de souci. Tu vois d'où ça peut venir?
Même erreur quand j'essaie d'ajouter un numéro sur une voie métrique
|
Super bien vu le deuxième, c'est corrigé ;) |
Context
Changement de la stack, on remplace mongodb par postgres grace a typeorm