Implements ProConnect #73
Conversation
| @@ -146,8 +147,13 @@ const Header = () => { | |||
| ))} | |||
| <Button | |||
| iconId="fr-icon-logout-box-r-line" | |||
| onClick={() => { | |||
| logout()(dispatch); | |||
| onClick={async () => { | |||
There was a problem hiding this comment.
On ne peut pas faire un lien qui tape le backend et que celui-ci fasse un joli HTTP_REDIRECT ?
Comme ça pas besoin de JS.
|
|
||
| const user = await userRepository.findOne('[email protected]'); | ||
| const user = await Users() |
There was a problem hiding this comment.
Je note, je vais corriger ça plus tard, en attendant ça me va.
|
|
||
| await Users().insert([ | ||
| { | ||
| id: uuidv4(), | ||
| email: '[email protected]', | ||
| email: 'admin@maestro.beta.gouv.fr', |
There was a problem hiding this comment.
haaa donc on a déjà une boite mail pour tout le domaine maestro.beta.gouv.fr ?
| return response.status(constants.HTTP_STATUS_OK).json(logoutUrl); | ||
| }; | ||
|
|
||
| export default { |
There was a problem hiding this comment.
C'est vraiment pas pratique le export default surtout pour du multiple export.
- Pour supprimer/ajouter une méthode, faut le modifier la méthode ET le bas du fichier
- Quand je suis sur une méthode, je ne peux pas savoir si elle est exporter ou pas. Je dois aller tout en bas du fichier pour le savoir
Si tu cherches sur le net, la recommandation actuelle est d'éviter les export default si c'est possible.
Exemple =>
Using default exports can make it harder to find things in your code and lead to inconsistencies. Instead, named exports make it easier to discover which members are being exported and what their corresponding names are, especially when using an IDE.
It's recommended to avoid using default exports and instead opt for named exports whenever possible. By doing so, you'll have a more organized and maintainable codebase that's easier to work with in the long run.
C'est pas le but de la PR, donc ne change pas. Mais sur les futurs devs, je pense qu'on doit de mettre à utiliser export const myFunction partout. Je suis bien entendu dispo pour en parler 😃
No description provided.