Append attachments to comments by flamingo-cant-draw ?? Pull Request #5143 ?? LemmyNet/lemmy

  • Andrew@piefed.social
    link
    fedilink
    English
    arrow-up
    2
    ·
    2 days ago

    How does Piefed handle image attachments, btw?

    For comments: not at all. If a Mastodon user tried to do what I did, with the inline image, nothing would show.

    We could do what I think you’ve done, and regex the details of the attachment into ! [] () Markdown and add it to the text. There’s also a DB relationship between comments and images that isn’t used, but could be, I suppose.

    I’ve never actually seen a Mastodon user try to add an image to something that ended up as a Lemmy comment, tbh, so it’s not something I’ve thought too much about.

    • flamingos-cant@feddit.uk
      link
      fedilink
      English
      arrow-up
      3
      ·
      edit-2
      2 days ago

      We could do what I think you’ve done, and regex the details of the attachment into ! [] ()

      To be clear, this pull request doesn’t use regex, it’s just JSON deserialisation and string interpolation.

      I’ve never actually seen a Mastodon user try to add an image to something that ended up as a Lemmy comment, tbh, so it’s not something I’ve thought too much about.

      The pull request actually includes one, the main KDE account tags [email protected] and includes pictures in their threads regularly. It’s just hard to tell from our side as you can’t see what’s missing.

      • Andrew@piefed.social
        link
        fedilink
        English
        arrow-up
        3
        ·
        2 days ago

        Ah, great, thank you. It’s been added as an Issue for PF now, with a link to this post, so that’ll be handy.

        (I was likely misusing the term ‘regex’).