Correctly type signature of executeCommand

mattermost / mattermost-redux

Redux for Mattermost

Apache License 2.0

200 stars 386 forks source link

Correctly type signature of executeCommand #1354

Closed hanzei closed 3 years ago

hanzei commented 3 years ago

Summary

executeCommand should have a proper typing with CommandArgs as the second argument

Ticket Link

https://mattermost.atlassian.net/browse/MM-32594

hmhealey commented 3 years ago

Funny. I just fixed the same thing in https://github.com/mattermost/mattermost-redux/pull/1355, although it was for a different action. Yours has a field that mine's missing, so I'll merge yours and update my branch accordingly.