-
Notifications
You must be signed in to change notification settings - Fork 10
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
Replaced by FedCM login-status API? #55
Comments
Based on what I heard from @johnwilander at TPAC (notes here), yes, I do believe that https://github.com/fedidcg/login-status is a more up to date explainer for this API than what's in this repo. I don't know if we want to point this repo to that one, or merge the text and move this repo to the FedID CG. But I do believe that:
@johnwilander did I get this right? i'd be happy to course correct and fix any misunderstanding or confusion I may have created. |
Let’s work on getting a PR on this spec since it’s the original proposal and has a lot of community input. |
Are you still happy with the explainer that we co-wrote at TPAC that we pasted here? https://github.com/fedidcg/login-status If so, I can send a PR to use this as the explainer for this repo. If not, can you send PRs with any further suggestions you may have? |
@johnwilander friendly ping? how do you want us to go about this? We recently chartered the FedID WG to include the Login Status API, so we are trying to figure out a few things:
|
Hi! I think it should remain its own spec, I want to remain as editor or coeditor, and we have some feedback on what’s in Chrome today since it deviates from the shape of the API we proposed in an unfortunate way. Specifically sending the status as a string. We’d like to discuss it at TPAC. |
Great! I'm breaking this up from the FedCM spec in this PR.
Great! I'll add you as the co-editor!
Can you expand on this a bit? We implemented what we discussed at TPAC last year here. Did chrome's implementation deviate from this? |
Ok, just to report back here, I just finished breaking the spec out of the FedCM spec and publishing this here as a co-editor: |
Is https://github.com/fedidcg/login-status replacing this proposal?
The text was updated successfully, but these errors were encountered: