transfem-org / Sharkey

🌎 A Sharkish microblogging platform 🚀
https://joinsharkey.org/
73 stars 19 forks source link

Including files tab contains posts that do not include files #118

Closed codingneko closed 10 months ago

codingneko commented 10 months ago

💡 Summary

The "Including files" tab in user notes contains posts that do NOT have a file because they reply to posts that do contain a file.

🥰 Expected Behavior

Only replies / posts that contain files should be shown, the post being replied to should not be a factor.

🤬 Actual Behavior

The Including Files tab is filled with replies to posts that contain a file, but that have no files, making the posts that actully have a file much harder to find.

📝 Steps to Reproduce

Access a profile from a user that has posted images or videos, and that has replied to posts with images or videos (most profiles fit this criteria), click on the "Notes" tab, and click on the "Including files" tab. Look at the posts that show up. Most of them do not have any files, they just reply to posts that do.

📌 Environment

2023.10.3.beta1

💻 Frontend

Currently on this, but affects all devices probably.

🛰 Backend (for server admin)

Self-hosted on bare-metal server running Ubuntu 22.04.3 LTS x86_64

codingneko commented 10 months ago

Acutally I'm not sure this is related to the original post having an image anymore... these do not have a file at all... I think the filter might just straight up not be working at all image screenshotted profile for reference: https://misskey.codingneko.com/@vaeleery@neovibe.app

dakkar commented 10 months ago

I think I fixed this, see https://github.com/transfem-org/Sharkey/pull/122