Skip to content
Snippets Groups Projects

[feat] gva: block

Merged Pascal Engélibert requested to merge tuxmain/duniter:gva-block into dev

Add GVA query for block.

Chez moi ça marche, par contre quand le bloc n'existe pas c'est le bloc par défaut qui est retourné. Que faut-il renvoyer dans ce cas ?

Pour des clients qui veulent lire la blockchain, ça serait utile de pouvoir demander plusieurs blocs avec pagination. Si c'est pas trop compliqué j'essaierai de faire ça.

Edited by Pascal Engélibert

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
    • Resolved by Éloïs

      quand le bloc n'existe pas c'est le bloc par défaut qui est retourné. Que faut-il renvoyer dans ce cas ?

      Il faut que tu retourne une Option, et que tu retourne None quand le block n'existe pas. Lors de la sérialisation en JSON les Option::None deviennent des null.

      ça serait utile de pouvoir demander plusieurs blocs avec pagination. Si c'est pas trop compliqué j'essaierai de faire ça.

      Exact cette requête devrait être paginée. La pagination c'est déjà un peu plus compliqué, on se programme une visio pour ça ? Où alors je lance une 2ème session de la formation GVA pour expliquer la pagination ? Ça me fais juste *** de créer un diapo, à voir si c'est faisable sans diapo !

  • Éloïs
  • added 2 commits

    • f1512300 - 1 commit from branch nodes/typescript:dev
    • dde90361 - [feat] gva: block

    Compare with previous version

  • Pascal Engélibert marked this merge request as ready

    marked this merge request as ready

  • added 1 commit

    Compare with previous version

  • Éloïs approved this merge request

    approved this merge request

  • Éloïs resolved all threads

    resolved all threads

  • Éloïs enabled an automatic merge when the pipeline for f0da0fab succeeds

    enabled an automatic merge when the pipeline for f0da0fab succeeds

  • Éloïs mentioned in commit f05a4c70

    mentioned in commit f05a4c70

  • merged

  • Please register or sign in to reply
    Loading