NethermindEth/starknet.go

Specific return type for BlockWithTxs and BlockWithTxHashes methods

Ouverte

#355 ouverte le 22 sept. 2023

Voir sur GitHub
 (1 commentaire) (0 réaction) (0 personne assignée)Go (140 forks)auto 404
help wanted

Métriques du dépôt

Stars
 (177 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

In https://github.com/NethermindEth/starknet.go/pull/354 a new type for blocks with only transaction hashes was created as requested on #347.

However, BlockWithTxs and BlockWithTxHashes methods still have as return type interface{} as these can return either a Block or a Pending Block in both cases. The idea is to replace interface{} with an specific type and do something similar as in TransactionByHash method (see here) with adaptTransaction

Guide contributeur

Direction de recherche
Étudiez la méthode `TransactionByHash` et sa fonction `adaptTransaction` dans `rpc/transaction.go` pour comprendre comment des types spécifiques sont retournés. Appliquez ensuite le même modèle aux méthodes `BlockWithTxs` et `BlockWithTxHashes` : définissez un nouveau type pouvant contenir à la fois les variantes block et pending block, similaire à l'utilisation du type `Transaction`. Vérifiez les types de bloc existants dans le code et créez un type de bloc adapté si nécessaire.
Stack technique
go
Domaine
backendapi
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GoType assertion in Go
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75