Closed cbiesinger closed 1 year ago
LGTM
Do we still actually need this explainer now that the spec is patched?
I think the explainer is still useful as a more concise description of the feature. (Note how the blink launch process needs explainers even when the spec PR has landed...)
Anyway, your suggestion is committed, so this is ready for merging.
Most notably, the signin_url field is renamed to login_url to match the spec.