Support for lbrynet daemon v0.30.0 #1998
Labels
No labels
accessibility
app-parity
area: creator
area: daemon
area: design
area: devops
area: discovery
area: docs
area: installer
area: internal
area: livestream
area: performance
area: proposal
area: reposts
area: rewards
area: search
area: security
area: subscriptions
area: sync
area: ux
area: viewer
area: wallet
BEAMER
channel
comments
community PR
consider soon
core team
css
dependencies
electron
Epic
feature request
first-timers-only
good first issue
hacktoberfest
help wanted
hub-dependent
icebox
Invalid
level: 0
level: 1
level: 2
level: 3
level: 4
merge when green
needs: exploration
needs: grooming
needs: priority
needs: repro
needs: tech design
notifications
odysee
on hold
playlists
priority: blocker
priority: high
priority: low
priority: medium
protocol dependent
recsys
redesign
regression
resilience
sdk dependent
Tom's Wishlist
trending
type: bug
type: discussion
type: improvement
type: new feature
type: refactor
type: task
type: testing
unplanned
windows
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: LBRYCommunity/lbry-desktop#1998
Loading…
Reference in a new issue
No description provided.
Delete branch "lbrynet30"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
@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)
@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.
I'll try finishing this up
@ -160,4 +160,3 @@
key="channel"
type="select"
tabIndex="1"
onChange={this.handleChannelChange}
What is this change fixing?
@ -160,4 +160,3 @@
key="channel"
type="select"
tabIndex="1"
onChange={this.handleChannelChange}
It wasn't doing anything so I removed it.