[feat] gva: blocks
Add paginated request blocks to GVA.
@librelois Ça marche ! Par contre :
- En mode DESC, faut-il que le cursor soit inclus ou exclus ?
- Je ne sais pas comment obtenir proprement le nombre de blocs dans la db. J'en aurais besoin pour définir
first_cursor_optdans un certain cas. (d'où letodo!) - (lignes 92-172) Pour éviter d'écrire 4 fois la même chose en fonction du type du range, peut-être qu'il vaut mieux utiliser
first_cursor_opt.unwrap_or_default(0)..last_cursor_opt.unwrap_or_default(last_block_number)? - Est-ce que j'utilise le bon type pour le numéro du bloc ?
- Faut-il écrire un test de la requête GVA ?
Merge request reports
Activity
En mode DESC, faut-il que le cursor soit inclus ou exclus ?
Le cursor doit toujours être inclus.
Je ne sais pas comment obtenir proprement le nombre de blocs dans la db. J'en aurais besoin pour définir first_cursor_opt dans un certain cas. (d'où le todo!)
méthode count(). Pense à regarder le fichier collection_ro comme je te l'ai déjà dit ;)
(lignes 92-172) Pour éviter d'écrire 4 fois la même chose en fonction du type du range, peut-être qu'il vaut mieux utiliser
first_cursor_opt.unwrap_or_default(0)..last_cursor_opt.unwrap_or_default(last_block_number)?Non il faut que
first_cursor_optetlast_cursor_optvalent None quand on demande tout.Ici pour réduire la duplication il te faut définir des variables
k_minetk_maxqui elles vaudrons ce que tu propose :)Est-ce que j'utilise le bon type pour le numéro du bloc ?
Oui
Faut-il écrire un test de la requête GVA ?
Oui
Edited by Éloïsadded 2 commits
added 3 commits
-
b630c0f8...f229f43d - 2 commits from branch
nodes/typescript:dev - d518a7c5 - [feat] gva: blocks
-
b630c0f8...f229f43d - 2 commits from branch
added 15 commits
-
d518a7c5...94c24755 - 14 commits from branch
nodes/typescript:dev - 6f7a0ae9 - [feat] gva: blocks
-
d518a7c5...94c24755 - 14 commits from branch
