I left a couple of comments but the PR looks good to me
Yes, this make sense, frontend will only use rows where
user_id is set.
Sure, I updated the test to set
nil first and then to a set date. It’s useful to check both scenarios, a user that never clears their log will always have this column set to