search

mattermost / mattermost-plugin-msteams

Other
12 stars 10 forks source link

MM-58436: Notifications review feedback #672

Closed lieut-data closed 1 month ago

lieut-data commented 1 month ago

Summary

This implements a number of pieces of design review feedback, most notably:

Ticket Link

Fixes: https://mattermost.atlassian.net/browse/MM-58436

lieut-data commented 1 month ago

@cwarnermm, I'd love to pick your brain on some of the text in this new "notifications-only" approach to the MS Teams plugin. What's the best way for me to consolidate the changes and show case them your way?

cwarnermm commented 1 month ago

@lieut-data - If there's a test environment available, that may be the easiest way to see the user experience end-to-end and provide input.

lieut-data commented 1 month ago

@cwarnermm, I've pushed all the outstanding changes to https://msteams-sync-test.test.mattermost.cloud. Maybe we can review in our planned 1:1?

lieut-data commented 1 month ago

@cwarnermm, heads up that I'm going to merge this alongside some other PRs and we can evaluate the whole thing together tomorrow :)