NethermindEth/starknet.go

Specific return type for BlockWithTxs and BlockWithTxHashes methods

Offen

#355 geöffnet am 22.09.2023

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)Go (140 Forks)auto 404
help wanted

Repository-Metriken

Stars
 (177 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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

Contributor Guide

Research-Richtung
Studiere die Methode `TransactionByHash` und ihre Funktion `adaptTransaction` in `rpc/transaction.go`, um zu verstehen, wie spezifische Typen zurückgegeben werden. Wende dann dasselbe Muster auf die Methoden `BlockWithTxs` und `BlockWithTxHashes` an: Definiere einen neuen Typ, der sowohl Block als auch Pending Block Varianten enthalten kann, ähnlich wie der Typ `Transaction` verwendet wird. Überprüfe die vorhandenen Blocktypen im Code und erstelle bei Bedarf einen angepassten Blocktyp.
Tech Stack
go
Domain
backendapi
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GoType assertion in Go
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75