serlo / serlo.org-legacy

Legacy implementation of https://serlo.org/
https://serlo.org/
Apache License 2.0
18 stars 3 forks source link

Review notification listeners #605

Closed inyono closed 3 years ago

inyono commented 3 years ago

Finally got a reproduce case for the saving issues in staging:

  1. Import staging database
  2. Start API locally (and add it to definitions.local.php)
  3. Try to save a new revision for /1855. Notice it takes quite a while. When logging, you'll notice the biggest factor are the notification listeners.

So we need to change that up. Some ideas:

604 deactivates the notification listeners for now. So to test it locally, you need to revert those changes.