Comments (8)
Thank you, @dipak-demansol! This context is very helpful. I agree with your assessment that we can remove the "try again" text. Let's go with your recommendation of "Failed to disconnect user. Connect the user account to Microsoft Teams."
from mattermost-plugin-msteams-meetings.
Pls assign This to me.
from mattermost-plugin-msteams-meetings.
before, when then user disconnect the account again then user was getting this validation message
and now From that message i removed err= and the message looks like this
@cwarnermm i did the changes on that message by removing that err= but still is there any improvement we can do? or that validation message is Look good to you?
from mattermost-plugin-msteams-meetings.
Thanks, @dipak-demansol, for reaching out. I appreciate it.
Let's update this message to: "Failed to disconnect user. Connect the user account to Microsoft Teams, then try again."
from mattermost-plugin-msteams-meetings.
Thanks, @dipak-demansol, for reaching out. I appreciate it.
Let's update this message to: "Failed to disconnect user. Connect the user account to Microsoft Teams, then try again."
Thank you for the reply, i have some thought about that message.
:- i suggest we should remove that "then try again" text because user is already not connected and as per this message we're saying that connect account then try again to disconnect, i think our validation message focus should be on to tell user to connect account not to try again for disconnection.
if you still Confirm that we should use the validation message that you provided then pls let me know, ill use that.
from mattermost-plugin-msteams-meetings.
Thanks, @dipak-demansol, for reaching out. I appreciate it.
Let's update this message to: "Failed to disconnect user. Connect the user account to Microsoft Teams, then try again."Thank you for the reply, i have some thought about that message. :- i suggest we should remove that "then try again" text because user is already not connected and as per this message we're saying that connect account then try again to disconnect, i think our validation message focus should be on to tell user to connect account not to try again for disconnection. if you still Confirm that we should use the validation message that you provided then pls let me know, ill use that.
from mattermost-plugin-msteams-meetings.
Thanks, @dipak-demansol! Can you help me understand the context of this message, please? What is the user doing when this message appears?
- who is doing the disconnecting?
- are they disconnecting themselves or someone else?
- what are they disconnecting someone from exactly?
Thanks!
from mattermost-plugin-msteams-meetings.
Thanks, @dipak-demansol! Can you help me understand the context of this message, please? What is the user doing when this message appears?
- who is doing the disconnecting?
- are they disconnecting themselves or someone else?
- what are they disconnecting someone from exactly?
Thanks!
@cwarnermm
:- who is doing the disconnecting?
A :- consider i'm the user and i'm disconnecting my self
are they disconnecting themselves or someone else?
A :- themselves
what are they disconnecting someone from exactly?
A :- they Connected with msteem plugin and User Disconnected themselves and user again using the same disconnect command to disconnect themselves so as per the old validation "we're telling to user that disconnect command failed, first connect the account to microsoft"
from mattermost-plugin-msteams-meetings.
Related Issues (20)
- Add warning when creating a meeting on channels HOT 5
- MS Teams call icon is only showing in the collapsed plugin list HOT 1
- MS Teams collision detection will throw an error if users mashes the Create New Meeting button
- Polish: Update Plugin configuration screen with some extra text and a link to documentation HOT 1
- Release v1.0.0 HOT 1
- Improve Installation & Configuration Docs HOT 2
- 400 on installing using binary HOT 4
- Add command autocomplete HOT 4
- Release v1.0.1
- Cross-plugin task: Enable the CircleCI "test" job in each plugin repo that has a webapp plugin
- Master branch is throwing an error on webapp plugin startup HOT 7
- There Should be a separate command for Connect. HOT 1
- slash Command improvement HOT 1
- in the desktop app msteem call UI is not same as Web-app UI. HOT 5
- MS Teams Calling shows plugin ID as App Bar tooltip HOT 2
- Intermittent issue where user see's an error and must click the connection link a 2nd time
- Release v2.1.0 HOT 3
- Installation docs still lists E20 license as a requirement HOT 2
- x509: certificate is not valid for any names, but wanted to match login.microsoftonline.com HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from mattermost-plugin-msteams-meetings.