Updating an existing claim breaks the redux store #186

Closed
opened 2017-06-02 13:27:10 +02:00 by 6ea86b96 · 3 comments
6ea86b96 commented 2017-06-02 13:27:10 +02:00 (Migrated from github.com)

The Issue

Steps to reproduce

  1. Publish a file
  2. Now publish another file at the same uri

Expected behaviour

It should update or replace the existing claim and fileinfo

Actual behaviour

We have 2 versions of the claim in the store, which breaks the downloaded/published pages and the new fileinfo isn't loaded.

## The Issue ### Steps to reproduce 1. Publish a file 2. Now publish another file at the same uri ### Expected behaviour It should update or replace the existing claim and fileinfo ### Actual behaviour We have 2 versions of the claim in the store, which breaks the downloaded/published pages and the new fileinfo isn't loaded.
alexliebowitz commented 2017-06-03 09:55:47 +02:00 (Migrated from github.com)

@kauffj: I could take this one. It follows on naturally from the pre-fill stuff. Plus I'll be mucking around in the Redux logic where it stores the list of files to diagnose the bug I mentioned in #142.

@kauffj: I could take this one. It follows on naturally from the pre-fill stuff. Plus I'll be mucking around in the Redux logic where it stores the list of files to diagnose the bug I mentioned in #142.
kauffj commented 2017-06-05 20:00:07 +02:00 (Migrated from github.com)

@alexliebowitz please do, and please mark yourself as assignee when you'd like to take ownership. Thanks!

@alexliebowitz please do, and please mark yourself as assignee when you'd like to take ownership. Thanks!
6ea86b96 commented 2017-06-08 17:35:53 +02:00 (Migrated from github.com)

I think there's a good chance this one has gone now that we're storing claims by claim_id in redux.

I think there's a good chance this one has gone now that we're storing claims by claim_id in redux.
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#186
No description provided.