search

mattermost-community / mattermost-plugin-agenda

Mattermost plugin to handle meeting agendas
Apache License 2.0
35 stars 21 forks source link

Remove command name check #31

Closed attilamolnar closed 4 years ago

attilamolnar commented 4 years ago

Summary

Remove checking whether the command name passed to the handler is what we registered.

Ticket Link

Discussed in https://github.com/mattermost/mattermost-plugin-agenda/pull/28#pullrequestreview-396465460

attilamolnar commented 4 years ago

Many (all?) other plugins have a similar check:

If it is indeed unnecessary I'll clean up the rest of the plugins as well.