Fix claim ID being null when reporting a claim that was not previously download. #1530
Conversation
DanielaNarvaez
changed the title
| @@ -12,6 +12,7 @@ const select = (state, props) => ({ | |||
| /* availability check is disabled due to poor performance, TBD if it dies forever or requires daemon fix */ | |||
| costInfo: makeSelectCostInfoForUri(props.uri)(state), | |||
| claimIsMine: makeSelectClaimIsMine(props.uri)(state), | |||
| claimId: props.claimId, | |||
There was a problem hiding this comment.
Since this is already being passed in from another component, we don't need to specify it here. This is only for selecting props from state.
|
Woohoo @Daninz, thanks for the contribution - glad you made it through your first PR! Normally we have a bot that reminds devs about updating the changelog, but it's disabled at the moment. Can you please do so at your earliest convenience? See previous entries for format...it's pretty straightforward. If you are interested in an LBC tip for your time, please see https://lbry.io/faq/tips We'll also be adding you to our developer mailing list so we can keep in touch (you can unsubscribe anytime!). |
|
Thanks @seanyesmunt for the review, i have updated the code as you requested. |
* Disable ads outside the US (again)
Closes issue: #1512 properly pass claim ID to the https://lbry.io/dmca?claim_id url, as @tzarebczan told me I will continue to work on #1425 but for now the fix closes #1512