Dave branch#1
Conversation
|
Hello @kessa99 , Great work. La branch a un conflit stp est ce que tu peut voir ca ? |
|
Ok je vais revoir cela |
|
c'est bon |
|
Hi @kessa99 , je vu le gros boulot qui est fait vraiment merci beaucoup. en faisant le review j'ai remarquer qu'il sera tres cool de faire de petit modifier et ouvrir un pr pour chaque modif. parce que avec tout en un vraiment c'est tres dificile de review ca proprepement sans faire d'erreur (: |
| raise HTTPException(status_code=409, detail="A speaker with this email already exists for this event") | ||
|
|
||
| data = {k: str(v) if v is not None and not isinstance(v, (bool, int, dict)) else v | ||
| for k, v in payload.items()} |
There was a problem hiding this comment.
Hello @kessa99 , merci beaucoup pour ta contribution.
Ici je trouve ce check n'ai plus necessaire puisse qu'on utilise deja Pydantic. la boucle va ajouter une charge de plus. au niveau du mini il aussi deja ces checker encore.
| async def update_speaker(db, speaker_id: str, payload: dict) -> dict: | ||
| await _get_or_404(db, speaker_id) | ||
| payload["updated_at"] = datetime.now(timezone.utc) | ||
| data = {k: str(v) if v is not None and not isinstance(v, (bool, int, dict)) else v |
| return str(rows[0]["id"]) | ||
|
|
||
|
|
||
| async def _get_or_404(db, speaker_id: str) -> dict: |
There was a problem hiding this comment.
j'aime cette approche. je trouvre que ca sera cool de le rendre module generale pour l'utiliser dans autre module
| @@ -0,0 +1,72 @@ | |||
| from datetime import datetime, timezone | |||
There was a problem hiding this comment.
ce fichier a toujours un conflit
2 participants
add new endpoint