You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The former gives all of the posts descendants, including children of child statuses and so on. The former seems to only return immediate descendants. Not only does this cover up a lot of discussion, but it also makes replying to descendants weird, in that it'll send and get recorded on the server side but then just won't render in Resin.
I tried looking into how PixelDroid deals with it but it looks like they have a bug that makes every post say "No comments found," as tested on pixelfed.social and pixelfed.de. I'm not sure if this is related somehow, but it might still be worth digging through their code base to get a better idea of how to properly approach this if needed and if possible.
This should also be manually confirmed using a terminal, just for the peace of mind. Needs more investigation
The text was updated successfully, but these errors were encountered:
Pixelfed seems to only return first-level descendants from its context
endpoint, thus after replies-to-replies are submitted, they are not
rendered. See #23
Compare these two requests (the second seems to require an Authorization header, just a heads up):
The former gives all of the posts descendants, including children of child statuses and so on. The former seems to only return immediate descendants. Not only does this cover up a lot of discussion, but it also makes replying to descendants weird, in that it'll send and get recorded on the server side but then just won't render in Resin.
I tried looking into how PixelDroid deals with it but it looks like they have a bug that makes every post say "No comments found," as tested on
pixelfed.social
andpixelfed.de
. I'm not sure if this is related somehow, but it might still be worth digging through their code base to get a better idea of how to properly approach this if needed and if possible.This should also be manually confirmed using a terminal, just for the peace of mind. Needs more investigation
The text was updated successfully, but these errors were encountered: