This pull request fixes an issue where unknown events to the router will register as a "close" event, since it is not pulling up with the logic in the if/elseif statements currently in place, so it'll think it's close.
This PR explicitly checks if the Merge Request and Issue is a close event, and will treat as such. If it doesn't recognize any of the other events that are currently in place, it will simply return.
matthew119427
commented
4 years ago
This pull request fixes an issue where unknown events to the router will register as a "close" event, since it is not pulling up with the logic in the if/elseif statements currently in place, so it'll think it's close. This PR explicitly checks if the Merge Request and Issue is a close event, and will treat as such. If it doesn't recognize any of the other events that are currently in place, it will simply
return.