lbryio / lbry-desktop

A browser and wallet for LBRY, the decentralized, user-controlled content marketplace.
https://lbry.tech
MIT License
3.56k stars 414 forks source link

Fix UX around publishing to already published URL #2882

Closed tzarebczan closed 4 years ago

tzarebczan commented 5 years ago

If you select a file and the auto-suggested name collides with an existing publish, it's not super clear that this new file will overwrite your previous publish. I think we should either disable the publish button and put the "edit this claim" text near it, or make the publish button say edit with a strong warning.

image

tzarebczan commented 4 years ago

A few users ran into this issue recently and overwrote other claims.

tzarebczan commented 4 years ago

A few users recently overwrote their claims. We should either improve this in the meantime by not allowing publish unless they click edit, or do https://github.com/lbryio/lbry-desktop/issues/3704

ceoger commented 4 years ago

I have also seen a few users complain about tier the last upload overwriting a previous upload.

My guess was they were probably editing a claim and ended up publishing a new file on the old claims. If that is the case, maybe we can have a warning after editing a claim where user selects a new file.

kauffj commented 4 years ago

(Closed this prematurely, but fix will be merged very soon)