Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

GetUnspentFromWallet panic's on unspent deposits #190

Open
hamdiallam opened this issue Mar 18, 2020 · 0 comments
Open

GetUnspentFromWallet panic's on unspent deposits #190

hamdiallam opened this issue Mar 18, 2020 · 0 comments
Labels
bug Something isn't working
Milestone
Version 1.0

Comments

@hamdiallam
Copy link
Member

Summary of bug

`(DataStore).GetUnspentFromWallet assumes that all unspent outputs are transaction outputs. When there's an unspent deposit, this causes plasmad to panic.

Expected behavior

GetUnspentFromWallet should return general output information. If wanted, the TxOuput or DepositOutput should be an additional call.

Steps to reproduce

Deposit into the smart contract, create and include-deposit transaction, then call plasmacli query info on the account that owns the deposit.

Versions/External Software

0.2.0-91-g8f898d9
@hamdiallam hamdiallam added the bug Something isn't working label Mar 18, 2020
@hamdiallam hamdiallam added this to the Version 1.0 milestone Mar 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Milestone
Version 1.0
Development

No branches or pull requests
1 participant
@hamdiallam