fix: use short form uri when preparingEdit so users know they are editing #1501

Merged
neb-b merged 1 commit from editing-claim-fix into master 2018-05-24 05:42:48 +02:00
neb-b commented 2018-05-23 06:36:05 +02:00 (Migrated from github.com)

Fixes https://github.com/lbryio/lbry-app/issues/1499
We store the short form of the uri being edited. That was causing an issue with properly prefilling claim edits because we were passing in the long form with a claim id. In the place that we were checking if the current uri is equal to the uri being edited, they were different so it wasn't working.

// check to see if you are editing
// editingUri === uri

// before
lbry://@channelName#xxx.../claim-name === lbry://@channelName/claim-name

Fixes https://github.com/lbryio/lbry-app/issues/1499 We store the short form of the uri being edited. That was causing an issue with properly prefilling claim edits because we were passing in the long form with a claim id. In the place that we were checking if the current uri is equal to the uri being edited, they were different so it wasn't working. ``` // check to see if you are editing // editingUri === uri // before lbry://@channelName#xxx.../claim-name === lbry://@channelName/claim-name ```
neb-b commented 2018-05-23 06:41:06 +02:00 (Migrated from github.com)

@skhameneh

@skhameneh
skhameneh (Migrated from github.com) approved these changes 2018-05-24 04:54:06 +02:00
Sign in to join this conversation.
No reviewers
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#1501
No description provided.