-
Notifications
You must be signed in to change notification settings - Fork 0
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/modification formulaire #298
Conversation
…ge into feat/campagnes-saisie-intentions
…ge into feat/campagnes-saisie-intentions
…ge into feat/campagnes-saisie-intentions
d43ed55
to
50b1841
Compare
…ge into feat/modification-formulaire
@@ -49,6 +49,13 @@ export interface DB extends Omit<DBSchema, "formationNonMaterializedView"> { | |||
>; | |||
}; | |||
} | |||
export interface DB extends Omit<DBSchema, "latestDemandeView"> { |
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 m'intérroge sur les multiples exports d'interface DB, on ne peut pas regrouper tout ça dans une seule ?
server/src/modules/core/usecases/getCampagnes/getCampagnes.route.ts
Outdated
Show resolved
Hide resolved
eb( | ||
sql`unaccent(${eb.ref("discipline.libelleDiscipline")})`, | ||
"ilike", | ||
`%${search_word |
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.
C'est pas la première fois qu'on a besoin de normaliser un texte de recherche, ça vaudrait le coup de créer une méthode globale qu'on pourrait réutiliser
server/src/modules/intentions/repositories/updateDemandeWithHistory.query.ts
Outdated
Show resolved
Hide resolved
server/src/modules/intentions/usecases/countDemandes/countDemandes.usecase.ts
Show resolved
Hide resolved
server/src/modules/intentions/usecases/deleteDemande/deleteDemande.route.ts
Outdated
Show resolved
Hide resolved
user: Pick<RequestUser, "id" | "role" | "codeRegion">; | ||
}) => { | ||
const demande = await deps.findOneDemande(id); | ||
const demande = await deps.findOneDemande(numero); | ||
if (!demande) throw Boom.notFound(); | ||
|
||
const scope = getPermissionScope(user.role, "intentions/ecriture"); |
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.
Cette règle de vérification pourrait potentiellement être faite coté DB avec des RLS
server/src/modules/intentions/usecases/getDefaultCampagne/getDefaultCampagne.route.ts
Outdated
Show resolved
Hide resolved
server/src/modules/intentions/usecases/getDefaultCampagne/getDefaultCampagne.usecase.ts
Outdated
Show resolved
Hide resolved
@@ -1,4 +1,6 @@ | |||
import * as uuid from "uuid"; | |||
|
|||
export const generateId = () => | |||
export const generateId = () => uuid.v4().replace(/-/g, ""); |
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.
utilise des ulid dans ce cas, tu pourras les ordonner de manière alphabhétique, chose qui n'est pas le cas la
...pped)/intentions/saisie/intentionForm/typeDemandeSection/capaciteSection/CapaciteSection.tsx
Show resolved
Hide resolved
...ns/saisie/intentionForm/typeDemandeSection/capaciteSection/CapaciteScolaireActuelleField.tsx
Outdated
Show resolved
Hide resolved
…ge into feat/modification-formulaire
Quality Gate failedFailed conditions |
Docker version obsolète reference : https://github.com/compose-spec/compose-spec/blob/master/04-version-and-name.md
Documentation capacité : https://www.notion.so/R-gles-de-validation-des-capacit-s-574f33e0ac0f427da8c53482ec1d0b16