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

WebAuthN Docs: add descriptions and details for PublicKey credential creation #24813

Conversation

jsmrcaga
Copy link
Contributor

Description

This Pull Request adds details on the CredentailsContainer page. More specifically, it adds descriptions for the possible options used under publicKey when requesting the creation of a new credential. These have been taken directly from the W3C specification.

Motivation

I have integrated WebAuthN three different times, and the three times I have needed to navigate between dev articles, blog posts, W3C docs and MDN. I would like other developers to be able to find the necessary information (or official links) faster and in a more straightforward manner.

Additional details

I have not been able to find more information on Federated and Password credential creation, although there is another PR for federated docs. I will certainly create a new PR with descriptions for the get method soon.

I'm also getting a formatting artifact when rendering locally because of the colon preceding the description. Please let me know if this is a local issue and if I need to take any steps to fix it.
Screenshot 2023-02-23 at 10 23 50 PM

Related issues and pull requests

@jsmrcaga jsmrcaga requested a review from a team as a code owner February 23, 2023 21:30
@jsmrcaga jsmrcaga requested review from sideshowbarker and removed request for a team February 23, 2023 21:30
@github-actions github-actions bot added the Content:WebAPI Web API docs label Feb 23, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Feb 25, 2023

Preview URLs

Flaws (7)

URL: /en-US/docs/Web/API/CredentialsContainer/create
Title: CredentialsContainer.create()
Flaw count: 7

  • macros:
    • /en-US/docs/Web/API/CredentialCreationOptions does not exist
    • /en-US/docs/Web/API/PasswordCredentialData does not exist
    • /en-US/docs/Web/API/CredentialData does not exist
    • /en-US/docs/Web/API/FederatedCredentialInit does not exist
    • /en-US/docs/Web/API/CredentialData does not exist
    • and 2 more flaws omitted
External URLs (1)

URL: /en-US/docs/Web/API/CredentialsContainer/create
Title: CredentialsContainer.create()

(comment last updated: 2023-02-25 10:16:12)

@sideshowbarker sideshowbarker merged commit 0bbaf29 into mdn:main Feb 25, 2023
@sideshowbarker
Copy link
Collaborator

Jo, thanks much, and congrats on landing your first docs change here — welcome aboard 🎉

@jsmrcaga jsmrcaga deleted the jo/add-webauthn-publickey-options-descriptions branch February 25, 2023 17:52
@jsmrcaga
Copy link
Contributor Author

@sideshowbarker thanks a lot!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:WebAPI Web API docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants