Skip to content

Conversation

@cbiesinger
Copy link
Collaborator

@cbiesinger cbiesinger commented Jun 24, 2025

@cbiesinger cbiesinger requested a review from npm1 June 24, 2025 20:38
1. Act as if the user had triggered the "Use another account" affordance
and initiate the [=IDP=] login flow as described above.

Note: Because multiple IDPs are only supported in passive mode, and
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm should this command have an IDP for logging in though? Otherwise how do you know which IDP to log in to for the dialog in 2668?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As the note says, there can only be one IDP

If we support the combination of use other account and multiple IDPs at some point, we can add an index parameter similar to what https://github.com/w3c-fedid/FedCM/pull/610/files was trying to do.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And the case on line 2668 is pre-existing, just reformatted basically

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.

2 participants