-
Notifications
You must be signed in to change notification settings - Fork 6
Documentation: Add reference docs for WellKnownWebAuthnView; create how-to guides section and add guide on how to configure for related origins #72
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
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #72 +/- ##
=======================================
Coverage 100.0% 100.0%
=======================================
Files 13 13
Lines 748 748
Branches 63 63
=======================================
Hits 748 748 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
0bc9e88
to
d2b3ef5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should also make some mention of browser support, as you need fairly recent versions of Chrome and Safari for this to work.
Notably, Firefox doesn't support this yet.
There are also important rules around the number of related origins, see this section: https://passkeys.dev/docs/advanced/related-origins/#how-it-works
8e4114e
to
8ca0784
Compare
…ne content in PR#73, thereby creating a how-to guide section; guide on how to configure for related origins; move the guides in Customizing behavior to the how-to guide section
8ca0784
to
0552e5e
Compare
Back from holidays, will add this to my review queue again 👍 |
…minor grammar issues in configure_related_origins.
This PR provides a reference documentation for
WellKnownWebAuthnView
.Related links