Skip to content
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

Change "children" prop to optional. #117

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

SubhanshuB
Copy link

Refactor the Microsoft SSO login functionality to provide the user with a choice between the default login button and the custom component. The current version of the application enforces the use of the custom component for Microsoft SSO login, despite the presence of a default button. This update removes this limitation and enables the user to select their preferred login method.

By providing the user with the option to choose between the default button and the custom component, we increase the flexibility and usability of the application. Users who prefer the default login button can continue to use it, while those who require the additional functionality of the custom component can select that instead.

This change is expected to improve the overall user experience and make the application more accessible to a wider audience.

Refactor the Microsoft SSO login functionality to provide the user with a choice between the default login button and the custom component. The current version of the application enforces the use of the custom component for Microsoft SSO login, despite the presence of a default button. This update removes this limitation and enables the user to select their preferred login method.

By providing the user with the option to choose between the default button and the custom component, we increase the flexibility and usability of the application. Users who prefer the default login button can continue to use it, while those who require the additional functionality of the custom component can select that instead.

This change is expected to improve the overall user experience and make the application more accessible to a wider audience.
@serhiileanylabs
Copy link

is it gonna be merged?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants