Giter Site home page Giter Site logo

Comments (7)

skobkin avatar skobkin commented on July 23, 2024 1

@prolibre

I think these duplicates are linked to a bug

Good catch.

I didn't know that he didn't have the moderation information

Here's how it works for links to new domains:

  1. User adds the link.
  2. User do not see anything.
  3. Administrator gets a notification about pending link domain review.
GIF with demonstration

P.S. I guess some of those duplicated links could be caused by #2988 and some by UI flaws.

from bookwyrm.

MaggieFero avatar MaggieFero commented on July 23, 2024 1

I know on my instance I've seen a bunch of what looks to be a user being frustrated and not just the bug causing duplicates, this warning sounds very nice!

from bookwyrm.

skobkin avatar skobkin commented on July 23, 2024

An example of such federated book:

image

from bookwyrm.

prolibre avatar prolibre commented on July 23, 2024

I think these duplicates are linked to a bug (when books circulate between instances). But that doesn't mean that the user shouldn't be warned (I didn't know that he didn't have the moderation information).

from bookwyrm.

skobkin avatar skobkin commented on July 23, 2024

And guess what happens if users tries to add link again?

image

from bookwyrm.

MaggieFero avatar MaggieFero commented on July 23, 2024

I went to see if I could handle this quickly, and there is a line in the FileLinkForm that I would expect to do this (added by Mouse two years ago, per git blame).

I confirmed on my own instance by deliberately adding two copies of a link to the last book I read. I never got this warning, and when I approved the domain both appeared. So it seems like this may just be a small bug?

Line 49 of https://github.com/bookwyrm-social/bookwyrm/blob/main/bookwyrm/forms/links.py is the exact warning I'd expect to see here, in case somebody else has a chance to get to it before I do!

from bookwyrm.

skobkin avatar skobkin commented on July 23, 2024

#3311 probably fixes accidental duplication. Not sure if it helps to know WHY the link is not being shown yet.

from bookwyrm.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.