Ghost/ghost/members-events-service
Simon Backx 94e85dc09e
Reduced webhook calls when updating last_seen_at for email opens (#16008)
refs https://ghost.slack.com/archives/C02G9E68C/p1670960248186789

This reverts a change that was made here:

f4fdb4fa6c (r93071549),
but it still moved the original code to a new location in the
LastSeenAtUpdater

It includes a new E2E test to make sure timezones are supported
correctly.

- By not using Bookshelf, we no longer fire webhook calls
- By not using the member repository, we don't fetch and update the
member model and the labels relation in a forUpdate transaction, which
caused deadlock issues on the labels/members_labels tables which were
hard to resolve. Until now I was unable to find the other conflicting
transaction that caused this deadlock. Moving to raw knex (instead of
Bookshelf) and only updating the last_updated_at column should remove
the deadlock issue.

This removed the test for the email service wrapper, since it started
failing for an unknown reason and the test didn't make much sense (was
added earlier only to bump test threshold).
2022-12-14 17:50:42 +01:00
..
lib Reduced webhook calls when updating last_seen_at for email opens (#16008) 2022-12-14 17:50:42 +01:00
test Reduced webhook calls when updating last_seen_at for email opens (#16008) 2022-12-14 17:50:42 +01:00
.eslintrc.js Added the members-events-service package 2022-02-28 14:42:17 +01:00
index.js Added the members-events-service package 2022-02-28 14:42:17 +01:00
package.json Updated @tryghost dependencies (#16005) 2022-12-14 11:18:55 +07:00