0xMiden / 0xMiden/rust-sdk

Add `TransactionRecords` to handle execution chain ordering

Aperta
#2,207 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
enhancement
Lingua principale
Rust
Stelle
78
Fork
129
Merge medio
4g 11h
PR unite (30g)
52

Descrizione

> I wonder if we should basically create a new `TransactionRecords` struct and make this iterator a responsibility of that struct. We can also make `RpcCllient::sync_transactions()` return that new struct.
>
> There might be other refactors we can do from this. In any case, we can probably tackle that in a separate PR. If you agree, we can also bundle my other comment in that PR as well.

_Originally posted by @igamigo in https://github.com/0xMiden/miden-client/pull/2164#discussion_r3313007354_

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Esamina la discussione della PR #2164 riguardante RpcClient::sync_transactions() e il suggerimento di creare una struttura TransactionRecords. Esamina il codice attuale di gestione delle transazioni per comprenderne la logica di ordinamento. L'obiettivo è progettare una nuova struttura che gestisca l'ordinamento della catena di esecuzione e rifattorizzare il metodo sync_transactions in modo che la restituisca.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Ambito
backend-api-design
Tipo di issue
Refactoring
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
45/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.