edit_note_change

edit_note_change.sql

CREATE TYPE edit_note_status AS ENUM ('deleted', 'edited');

CREATE TABLE edit_note_change
(
    id                  SERIAL, -- PK
    status              edit_note_status,
    edit_note           INTEGER NOT NULL, -- references edit_note.id
    changing_editor 	INTEGER NOT NULL, -- references editor.id
    change_time         TIMESTAMP WITH TIME ZONE DEFAULT NOW(),
    old_note            TEXT NOT NULL,
    new_note            TEXT NOT NULL,
    reason              TEXT NOT NULL DEFAULT '',
);

Would it make sense to allow reason to be NULL for cases where no reason was provided? Or is it better to just have an empty string?

I'd rather have an empty string, but I'll change to TEXT NOT NULL DEFAULT ''

Maybe change_by_admin ought to be the ID of the editor who changed the note instead? Since it might be useful sometimes to know which admin changed something.

Ok, we considered that as an option but wasn't sure if needed, but I guess it can't hurt :)