Renaming discourse_voting to topic_voting since there are two forms of voting in Discourse - posts and topics.
This PR also moves a OnceOff into a post migration. The post migration will be executed, but should ideally be a no-op. This allows us to not have to maintain the OnceOff as it had to be modified before with a previous migration. I considered removing this file altogether, but I don't think there is anything negative from just converting it into a migration, and it might be useful in the unlikely scenario that a forum from the past has never ran the OnceOff before.
Previously, we were deleting the `enable_topic_voting` custom field param
and doing the staff logging in this plugin. This has a problem though, it
was being done in the model's create/destroy hooks and was missing the
acting user (so the logs would report the change as `system`).
This PR keeps the custom field param and relies on core to do the staff
action logging. This change also results in the custom field being set
in the DB when it is enabled even though we don't use it in the serializer.
Natalie Tay
Loïc Guitaut
Penar Musaraj
Faizaan Gagan
Krzysztof Kotlarek
Renato Atilio
Alan Guo Xiang Tan
Bianca Nenciu
David Taylor
Natalie Tay