credential-handler / authn.io

Credential Mediator Polyfill
https://github.com/w3c-ccg/credential-handler-api
Other
44 stars 8 forks source link

Consider whether to attempt to load `manifest.webmanifest` in addition to `manifest.json` #159

Closed dlongley closed 9 months ago

dlongley commented 9 months ago

See issue: https://github.com/credential-handler/credential-handler-polyfill/issues/41

Relevant line: https://github.com/credential-handler/authn.io/blob/v7.0.0/web/mediator/manifest.js#L23

If we're going to support the additional name, we should try to load them both in parallel to save time and will need to figure out which to prefer if both are present.

Other information on this in the ecosystem:

https://github.com/w3c/manifest/issues/689 https://developer.mozilla.org/en-US/docs/Web/Manifest

dlongley commented 9 months ago

Addressed by #160.