[MM-58290] Add UI to select notification preference. - Githubissues

mattermost / mattermost-plugin-msteams

MS Teams plugin for Mattermost

Other

13 stars 11 forks source link

[MM-58290] Add UI to select notification preference. #657

Closed JulienTant closed 4 months ago

JulienTant commented 4 months ago

Summary

This PR:

Adds in the redux store info about the user current connection status.
Adds the notifications preference in the user settings.
Use the new user current connection status to dynamically enable/disable the user preference options.
change some of the connect api logic to allow the connect button in the preference to work!

What is not in this PR be worth considering for future PRs?

Enabling/Disabling each configuration based on the current feature set of the plugin. Example: if selective sync is disabled, do we want to also disable (or not show) platform selection?

Ticket Link

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

JulienTant commented 4 months ago

@lieut-data Before working on the UI for the notification preference, I worked on having the connection status of the user in redux. Can you take a look and let me know if you see something I should fix at this stage?