chore(schoolbook): ENABLING-421 schoolbook module fetch update #230
Conversation
vmourot
left a comment
vmourot
left a comment
There was a problem hiding this comment.
Attention à ne pas modifier les signatures des fonctions si possible.
Le premier argument est en général la session, même si rarement utilisé en pratique.
francoisTouchardEdifice
force-pushed
the
feat/schoolbook-module-fetch-update
branch
2 times, most recently
from
1149114 to
366da4b
Compare
| const api = `/schoolbook/list`; | ||
| const body = JSON.stringify({ filter: 'Any', page }); | ||
| const entcoreTeacherWordList = (await signedFetchJson(`${session?.platform!.url}${api}`, { | ||
| const entcoreTeacherWordList = await sessionFetch.json<IEntcoreTeacherWordList>(`${session?.platform!.url}${api}`, { |
There was a problem hiding this comment.
Retirer session?.platform!.url car sessionFetch le gère automatiquement
| const api = `/schoolbook/relation/acknowledge/${wordId}/${studentId}`; | ||
| return signedFetchJson(`${session?.platform!.url}${api}`, { | ||
|
|
||
| return sessionFetch.json<Promise<{ id: number }>>(`${session?.platform!.url}${api}`, { |
There was a problem hiding this comment.
Retirer session?.platform!.url car sessionFetch le gère automatiquement
There was a problem hiding this comment.
- Pas besoin de typer avec Promise car la fonction est asynchrone
There was a problem hiding this comment.
ok fait
| const api = `/schoolbook/delete/${wordId}`; | ||
| return signedFetchJson(`${session?.platform!.url}${api}`, { | ||
|
|
||
| return sessionFetch.json<Promise<{ rows: number }>>(`${session?.platform!.url}${api}`, { |
There was a problem hiding this comment.
Retirer session?.platform!.url car sessionFetch le gère automatiquement
There was a problem hiding this comment.
- Pas besoin de typer avec Promise car la fonction est asynchrone
| const body = JSON.stringify({ studentId, text }); | ||
| return signedFetchJson(`${session?.platform!.url}${api}`, { | ||
|
|
||
| return sessionFetch.json<Promise<{ response_id: number }>>(`${session?.platform!.url}${api}`, { |
There was a problem hiding this comment.
Idem que pour les messages précédents
| const api = `/schoolbook/word/resend/${wordId}`; | ||
| return signedFetchJson(`${session?.platform!.url}${api}`, { | ||
|
|
||
| return sessionFetch.json<Promise<{ count: number; word_id: string }>>(`${session?.platform!.url}${api}`, { |
There was a problem hiding this comment.
Idem que pour les messages précédents
| const body = JSON.stringify({ text }); | ||
| return signedFetchJson(`${session?.platform!.url}${api}`, { | ||
|
|
||
| return sessionFetch.json<Promise<{ rows: number }>>(`${session?.platform!.url}${api}`, { |
There was a problem hiding this comment.
Idem que pour les messages précédents
francoisTouchardEdifice
force-pushed
the
feat/schoolbook-module-fetch-update
branch
2 times, most recently
from
701376b to
1fc1aba
Compare
francoisTouchardEdifice
changed the base branch from
dev/1.16.2
to
dev/1.16.3
francoisTouchardEdifice
force-pushed
the
feat/schoolbook-module-fetch-update
branch
from
1fc1aba to
0801dca
Compare
francoisTouchardEdifice
force-pushed
the
feat/schoolbook-module-fetch-update
branch
from
0801dca to
379e27c
Compare
|
francoisTouchardEdifice
deleted the
feat/schoolbook-module-fetch-update
branch
No description provided.