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.

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

Cannot send tip, transaction failed #917

Closed
tzarebczan opened this issue Jan 1, 2018 · 2 comments
Closed

Cannot send tip, transaction failed #917

tzarebczan opened this issue Jan 1, 2018 · 2 comments
Assignees
Labels
type: bug Existing functionality is wrong or broken

Comments

@tzarebczan
Copy link
Contributor

The Issue

Looks like this 7cd25e7#diff-3382ad8a365bcbe5fdac5b0508c405c7R158 recent code change broke the tip function. It should be claim_id and not claimID.

This is the current error:

 "error": {
    "code": -32602,
    "message": "Extraneous parameters for wallet_send command: claimId"
  },
  "id": 113,
  "jsonrpc": "2.0"

being passed as:

"wallet_send"
params:
{claimId: "008401a48cecef2a13cf5376b696169bb36b18c6", amount: 1}

Steps to reproduce

  1. Try to support a claim

Expected behaviour

Support goes through

Actual behaviour

Support fails to send

System Configuration

  • LBRY Daemon version:0.18.0
  • LBRY App version: 0.19.3
  • LBRY Installation ID:
  • Operating system:

Anything Else

Screenshots

@tzarebczan tzarebczan added the type: bug Existing functionality is wrong or broken label Jan 1, 2018
@liamcardenas
Copy link
Contributor

@IGassmann can you look into this?

IGassmann added a commit that referenced this issue Jan 2, 2018
Fix cannot send tip, transaction failed (#917)
@tzarebczan
Copy link
Contributor Author

Tested the fix, LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Existing functionality is wrong or broken
Projects
None yet
Development

No branches or pull requests

4 participants