tari icon indicating copy to clipboard operation
tari copied to clipboard

Cleanup wallet OMS output import

Open SWvheerden opened this issue 2 years ago • 1 comments

Currently, the wallet has 5 import requests for importing outputs

AddOutput((Box<UnblindedOutput>, Option<SpendingPriority>)),
    AddRewindableOutput((Box<UnblindedOutput>, Option<SpendingPriority>, Option<RewindData>)),
    AddOutputWithTxId((TxId, Box<UnblindedOutput>, Option<SpendingPriority>)),
    AddRewindableOutputWithTxId((TxId, Box<UnblindedOutput>, Option<SpendingPriority>, Option<RewindData>)),
    AddUnvalidatedOutput((TxId, Box<UnblindedOutput>, Option<SpendingPriority>)),

We can combine them into a single request AddUnvalidatedOutputand make the other data options. We only need to add invalidated outputs as the validation service will be triggered as soon as the output is added which will validate the output.

SWvheerden avatar Dec 06 '22 10:12 SWvheerden