Feat/information form#103
Conversation
| } | ||
| }; | ||
|
|
||
| export const createUserContactInformation: AppRequestHandler<UserContactInformation> = async (req, res) => { |
There was a problem hiding this comment.
Les types des controllers seront à décaller dans des DTOs
| } | ||
| }; | ||
|
|
||
| export const submitVssQuestionnaire: AppRequestHandler<user_service.VssSubmissionPayload> = async (req, res) => { |
There was a problem hiding this comment.
Pareil ça devrait venir du DTO et pas du service (un type dans le service c moche)
|
|
||
| // Admin routes | ||
| userRouter.get('/admin/getusersbypermission', checkRole('Admin', []), userController.getUsersByPermission); | ||
| userRouter.post('/admin/user', checkRole('Admin', []), userController.adminCreateUser); |
There was a problem hiding this comment.
Tu m'expliques pourquoi tu supprimes mes features ?
Je pense petite erreur lors des merge conflicts ;)
| user_id: integer('user_id') | ||
| .primaryKey() | ||
| .references(() => userSchema.id, { onDelete: 'cascade' }), | ||
| urgency_contact_name: text('urgency_contact_name'), |
There was a problem hiding this comment.
emergency serait pas plus correct niveau anglais ?
There was a problem hiding this comment.
qu'il est fort en anglais le president (mais oui ducoup)
| .primaryKey() | ||
| .references(() => userSchema.id, { onDelete: 'cascade' }), | ||
| urgency_contact_name: text('urgency_contact_name'), | ||
| urgency_contact_phone: text('urgency_contact_phone'), |
| fetchOnboardingStatus(); | ||
| }, []); | ||
|
|
||
| const fetchOnboardingStatus = async () => { |
There was a problem hiding this comment.
Il faut faire un composant <OnboardNeeded type="Nouveau"> par exemple pour filtrer l'affichage des différents composants, bien plus lisible selon moi que le code dupliqué sur chaque composant.
On peut lui fournir du texte pour les messages d'erreurs du type "billetterie soirée d'intégration" à ajouter après les messages d'erreur.
| fetchOnboardingStatus(); | ||
| }, []); | ||
|
|
||
| const fetchOnboardingStatus = async () => { |
| setMenuOpen(false); | ||
| }, [pathname]); | ||
|
|
||
| const fetchOnboardingStatus = async () => { |
There was a problem hiding this comment.
peut être passer par un statut global pour éviter les requêtes multiples sur le statut onboarding (autre PR selon moi)
Co-authored-by: Arthur Dodin <52950784+tuturd@users.noreply.github.com>
Co-authored-by: Arthur Dodin <52950784+tuturd@users.noreply.github.com>
Co-authored-by: Arthur Dodin <52950784+tuturd@users.noreply.github.com>
Remi-pcn
left a comment
Remi-pcn
left a comment
There was a problem hiding this comment.
A part les 2/3 points notés par arthur le reste peut attendre pour etre modifie. Sinon lgtm j'ai pas vu de dinguerie
| user_id: integer('user_id') | ||
| .primaryKey() | ||
| .references(() => userSchema.id, { onDelete: 'cascade' }), | ||
| urgency_contact_name: text('urgency_contact_name'), |
There was a problem hiding this comment.
qu'il est fort en anglais le president (mais oui ducoup)
| @@ -0,0 +1,4 @@ | |||
| export type UserContactInformation = { | |||
| urgency_contact_name: string; | |||
| urgency_contact_phone: string; | |||
There was a problem hiding this comment.
Pareil qu'avant ducoup, plutot "emergency"
3 participants
Uh oh!
There was an error while loading. Please reload this page.