Fix #250

• Fix #250 (closed) by adding the RPC module for Grandpa.

Note: proveFinality call will only work after the first authority set change.

assigned to @bgallois

changed milestone to %runtime-803

added RN-silent label

changed title from Draft: Fix {-https://git.duniter.org/nodes/rust/duniter-v2s/-/issues/-}250 to Draft: Fix #250

changed the description

added 1 commit

• a8c41d5a - add grandpa rpc

Compare with previous version

changed the description

added 1 commit

• 3d00bcbb - add grandpa rpc

Compare with previous version

changed the description

added 1 commit

• a55f5b4d - add grandpa rpc

Compare with previous version

marked this merge request as ready

requested review from @HugoTrentesaux

added 2 commits

• 7fe42a23 - 1 commit from branch master
• b27ee1da - add grandpa rpc

Compare with previous version

added RN-binary label

removed RN-silent label

Ça me semble bien, je confirme que j'ai bien les appels RPC grandpa (les mêmes que polkadot en tout cas) :

Juste par curiosité sur la méthode : comment as-tu procédé ? Un diff sur le node template ou sur un autre dépôt ? As-tu vu à l'occasion d'autres points qui mériteraient un ticket ? J'étais arrivé à #250 (closed) parce que ça m'a manqué pour débugger, mais si d'autres parties de l'API peuvent être ajoutées facilement, ça pourrait être bon à prendre !

Le template Solochain du SDK n'inclut pas le RPC pour GRANDPA. J'ai examiné la configuration des parachains qui l'ont et je l'ai adaptée à notre architecture ainsi qu'à notre version (la version 1.16.0 introduit déjà des changements au niveau des RPC). Ensuite, j'ai recherché dans les issues du Polkadot SDK pour comprendre pourquoi proveFinality ne fonctionnait pas "directement".

Après vérification, je pense que l'on a l'ensemble de l'API.

enabled an automatic merge when the pipeline for b27ee1da succeeds

merged

mentioned in commit 5628f095