When a user marks notifications as read (automatically) this will load activity entities one by one. Afterwards their status is changed one by one and caches are cleared for these entities.

We should at least load all activity entities at once.

Comments

bramtenhove created an issue. See original summary.

bramtenhove’s picture

bramtenhove CreditAttribution: bramtenhove for Open Social commented
Status: Active » Needs review

Here's a pull request that improves the code a little bit.

https://github.com/goalgorilla/open_social/pull/1384/files

jaapjan credited RobertRagas.

jaapjan’s picture

jaapjan CreditAttribution: jaapjan as a volunteer and at Open Social commented
Status: Needs review » Fixed

Merged to 5.x and 6.x, thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.