Add `TransactionRecords` to handle execution chain ordering
- 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