New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Manage linked accounts UI #23355
Manage linked accounts UI #23355
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
import ReactDOM from 'react-dom'; | ||
import ManageLinkedAccounts from './ManageLinkedAccounts'; | ||
|
||
export default class ManageLinkedAccountsController { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One wonders if we need this layer at all... but I guess we won't know until we've actually added behaviors.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, i'm thinking this will end up managing the overall state (which providers you're connected to, what should happen when you try to connect/disconnect from each provider, etc), but that might end up not being true
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
woops, i wasn't paying attention and was referring to something else ^^^
This adds the
ManageLinkedAccounts
component, currently with hard-coded emails and emptyonClick
handlers for each oauth provider.Examples
When you hover over a disabled "disconnect" button:
Note: I ended up not displaying the FontAwesome icons for each provider because the Microsoft logo wasn't available (maybe due to the version we are on? i didn't look into it much), but I can look into that once we've got more functionality finished for this feature.