Api endpoint for listing read posts #4934

[leoseg]

Added the read only flag to the post list endpoint and a unittest

[phiresky]

You probably want an index on post_actions (person_id, read) where read is not null to make this query performant

[leoseg]

@phiresky this index is already existing, created in the migration smoosh-tables-together from 2024-11-10

[phiresky]

which one do you mean?

[leoseg]

In migrations/2024-11-10-134311_smoosh-tables-together/up.sql in line 294 there is an index named 'idx_post_actions_read_not_null' created on (person_id, post_id) where read is not null. I also can see it in my local deployment db

[phiresky]

Yes. I do not think that index is adequate. See also #4459 (comment)

[leoseg]

ah sorry read it wrong, i added the index to the pull request

[leoseg]

Hi woodpecker failed on one of the federated api tests but in my local setup they ran trough, did you have this issue before ?

[Nutomic]

Hi woodpecker failed on one of the federated api tests but in my local setup they ran trough, did you have this issue before ?

Yes this sometimes happens and then fixes itself again, very tricky to debug. Ive restarted the build, you can also restart it yourself with git commit --allow-empty -m ci and git push.