Support for lbrynet daemon v0.30.0 #1998

Merged
eukreign merged 3 commits from lbrynet30 into master 2018-10-26 20:44:08 +02:00
eukreign commented 2018-10-03 23:53:41 +02:00 (Migrated from github.com)

@skhameneh This is ready for a review. @akinwale I would appreciate your eyes too to see if I missed anything else needed for v30.

Editing claims aren't working until https://github.com/lbryio/lbry/issues/1555 is merged. I think it will work as is. I would like to merge it soon so we can get an RC out and begin testing.

I was able to remove a lot of logic for tracking pending publishes since we can use the new confirmations field. It makes things a lot simpler.

Requires https://github.com/lbryio/lbry-redux/pull/79 and https://github.com/lbryio/lbryinc/pull/11 but you can run this branch as is (i'm just pointing at the commits from my PR)

@skhameneh This is ready for a review. @akinwale I would appreciate your eyes too to see if I missed anything else needed for v30. Editing claims aren't working until https://github.com/lbryio/lbry/issues/1555 is merged. I _think_ it will work as is. I would like to merge it soon so we can get an RC out and begin testing. I was able to remove a lot of logic for tracking pending publishes since we can use the new `confirmations` field. It makes things a lot simpler. Requires https://github.com/lbryio/lbry-redux/pull/79 and https://github.com/lbryio/lbryinc/pull/11 but you can run this branch as is (i'm just pointing at the commits from my PR)
akinwale (Migrated from github.com) reviewed 2018-10-03 23:53:41 +02:00
akinwale commented 2018-10-10 13:37:09 +02:00 (Migrated from github.com)

@eukreign You can make use of
parseFloat(amount).toFixed(precision || 1).replace(/\.?0+$/, '');
This is with the assumption that the amount passed is a valid numeric value.

@eukreign You can make use of `parseFloat(amount).toFixed(precision || 1).replace(/\.?0+$/, '');` This is with the assumption that the amount passed is a valid numeric value.
neb-b commented 2018-10-22 19:05:16 +02:00 (Migrated from github.com)

I'll try finishing this up

I'll try finishing this up
skhameneh (Migrated from github.com) approved these changes 2018-10-26 19:47:30 +02:00
@ -160,4 +160,3 @@
key="channel"
type="select"
tabIndex="1"
onChange={this.handleChannelChange}
skhameneh (Migrated from github.com) commented 2018-10-26 19:46:06 +02:00

What is this change fixing?

What is this change fixing?
neb-b (Migrated from github.com) reviewed 2018-10-26 19:48:11 +02:00
@ -160,4 +160,3 @@
key="channel"
type="select"
tabIndex="1"
onChange={this.handleChannelChange}
neb-b (Migrated from github.com) commented 2018-10-26 19:48:11 +02:00

It wasn't doing anything so I removed it.

It wasn't doing anything so I removed it.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lbry-desktop#1998
No description provided.