FEATURE: Navigate to an approved queued topic from the review queue. (PR #12841)

Admins can visit an approved queued topic from the review queue by clicking their title. We no longer store the created post and topic ids in the reviewable’s payload object. Instead, we set the topic_id and target_id attributes.

GitHub

I would avoid reaching into active record here and just hard code the status, makes the migration less brittle.

I would just make this non reversible. no need to have a down migration here. It is not really that reversible anyway, model will create new things afterwards.

maybe also for extra safty look for topic_id IS NULL and target_id IS NULL ? Maybe to limit the risks here we just forgo the migration altogether? on the fence on that but the migration parses and casts so there is an element of risk.

I’m merging this post-release.