webofneeds
webofneeds copied to clipboard
Too many createWonMessage methods
Describe the issue
Most actions in won-bot have a private or protected createWonMessage
method. Additionally, there is a deprecated createWonMessage
method in BotActionUtils. Instead of around 30 separate private methods, there should be a single utility method available to all bot actions.
This is currently not a bug, but may lead to unexpected behaviour if one or more methods are not updated in the future. Additionally, the code is harder to parse as many of these methods feature slightly different signatures.
removed the good first issue label, this is def. not for anybody who starts contributing to our repo... refactoring this will make people gouge their eyes out...