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

Add new spec: Login Status API #1608

Closed
Elchi3 opened this issue Dec 16, 2024 · 2 comments · Fixed by #1609
Closed

Add new spec: Login Status API #1608

Elchi3 opened this issue Dec 16, 2024 · 2 comments · Fixed by #1609
Labels
new spec Suggestion to add a new spec

Comments

@Elchi3
Copy link

Elchi3 commented Dec 16, 2024

URL

https://w3c-fedid.github.io/login-status/

Rationale

The spec got broken out in w3c-fedid/FedCM#642

In BCD we use:
https://w3c-fedid.github.io/FedCM/#dom-navigatorlogin-setstatus
and now we want to use:
https://w3c-fedid.github.io/login-status/#dom-navigatorlogin-setstatus

But that ED seems not present in browser-specs due to the TR still duplicating things?

"https://w3c-fedid.github.io/login-status/": {
"comment": "FedCM still contains the Login Status API as sub-section for now (and TR link actually targets it, which will likely confuse tooling)",
"lastreviewed": "2024-11-01"
}

Additional properties

{}
@Elchi3 Elchi3 added the new spec Suggestion to add a new spec label Dec 16, 2024
@browser-specs-bot
Copy link
Collaborator

Changes to index.json

This update would trigger the following changes in index.json:

Add spec (1)
{
  "url": "https://w3c-fedid.github.io/login-status/",
  "seriesComposition": "full",
  "shortname": "login-status",
  "series": {
    "shortname": "login-status",
    "currentSpecification": "login-status",
    "title": "Login Status API",
    "shortTitle": "Login Status API",
    "nightlyUrl": "https://w3c-fedid.github.io/login-status/"
  },
  "organization": "W3C",
  "groups": [
    {
      "name": "Federated Identity Working Group",
      "url": "https://www.w3.org/groups/wg/fedid/"
    }
  ],
  "nightly": {
    "url": "https://w3c-fedid.github.io/login-status/",
    "status": "Editor's Draft",
    "alternateUrls": [],
    "repository": "https://github.com/w3c-fedid/login-status",
    "sourcePath": "spec/index.bs",
    "filename": "index.html"
  },
  "title": "Login Status API",
  "source": "spec",
  "shortTitle": "Login Status API",
  "categories": [
    "browser"
  ],
  "standing": "good"
}

Tests

These changes look good! 😎

@tidoust
Copy link
Member

tidoust commented Dec 16, 2024

Yes, we postponed inclusion to avoid creating duplicate definitions for terms defined in the Login Status API (and sub-section of FedCM). Also, I thought code would also pick up the latest published version link from the Login Status API draft, which links to FedCM.

Now, the login status section was removed from the Editor's Draft of FedCM, which should be OK enough to avoid major cross-reference issues, and it turns out that the code does not trust the latest published version link in any case, so won't produce any incorrect release information. In other words, if you need it, it seems fine to add it.

@browser-specs-bot pr

github-actions bot pushed a commit that referenced this issue Dec 16, 2024
Close #1608, adding the suggested spec to the list.

### Changes to `index.json`
This update would trigger the following changes in `index.json`:

<details><summary>Add spec (1)</summary>

```json
{
  "url": "https://w3c-fedid.github.io/login-status/",
  "seriesComposition": "full",
  "shortname": "login-status",
  "series": {
    "shortname": "login-status",
    "currentSpecification": "login-status",
    "title": "Login Status API",
    "shortTitle": "Login Status API",
    "nightlyUrl": "https://w3c-fedid.github.io/login-status/"
  },
  "organization": "W3C",
  "groups": [
    {
      "name": "Federated Identity Working Group",
      "url": "https://www.w3.org/groups/wg/fedid/"
    }
  ],
  "nightly": {
    "url": "https://w3c-fedid.github.io/login-status/",
    "status": "Editor's Draft",
    "alternateUrls": [],
    "repository": "https://github.com/w3c-fedid/login-status",
    "sourcePath": "spec/index.bs",
    "filename": "index.html"
  },
  "title": "Login Status API",
  "source": "spec",
  "shortTitle": "Login Status API",
  "categories": [
    "browser"
  ],
  "standing": "good"
}
```
</details>

### Tests
These changes look good! 😎
tidoust pushed a commit that referenced this issue Dec 16, 2024
Close #1608, adding the suggested spec to the list.

### Changes to `index.json`
This update would trigger the following changes in `index.json`:

<details><summary>Add spec (1)</summary>

```json
{
  "url": "https://w3c-fedid.github.io/login-status/",
  "seriesComposition": "full",
  "shortname": "login-status",
  "series": {
    "shortname": "login-status",
    "currentSpecification": "login-status",
    "title": "Login Status API",
    "shortTitle": "Login Status API",
    "nightlyUrl": "https://w3c-fedid.github.io/login-status/"
  },
  "organization": "W3C",
  "groups": [
    {
      "name": "Federated Identity Working Group",
      "url": "https://www.w3.org/groups/wg/fedid/"
    }
  ],
  "nightly": {
    "url": "https://w3c-fedid.github.io/login-status/",
    "status": "Editor's Draft",
    "alternateUrls": [],
    "repository": "https://github.com/w3c-fedid/login-status",
    "sourcePath": "spec/index.bs",
    "filename": "index.html"
  },
  "title": "Login Status API",
  "source": "spec",
  "shortTitle": "Login Status API",
  "categories": [
    "browser"
  ],
  "standing": "good"
}
```
</details>

### Tests
These changes look good! 😎

Co-authored-by: browser-specs-bot <>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new spec Suggestion to add a new spec
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants