Skip to content

refactor notifications, add timestamp

Emma requested to merge refactor-notifications into main

This fixes a few things:

  • Changes type of notifications.id to UUID from INTEGER. Submissions and comments both have BIGINT IDs, making the size of the notification column inadequate, and since nothing references notifications, it was "easy" to switch to UUIDs.

  • Adds a timestamp column for ordering, which is also a prerequisite for a planned feature.

  • Moves persistence handling out of controller, into manager class.

  • Adds a few more assertions to make cause of failure clear in related integration tests.

Edited by Emma

Merge request reports