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

Update signature counters section. #1390

Merged
merged 2 commits into from Apr 2, 2020
Merged

Update signature counters section. #1390

merged 2 commits into from Apr 2, 2020

Conversation

agl
Copy link
Contributor

@agl agl commented Mar 18, 2020

This section did not reflect the specified behaviour for signature
counters and did not mention that they are returned in makeCredential
responses too. See linked bug for details.

Fixes #1370


Preview | Diff

This section did not reflect the specified behaviour for signature
counters and did not mention that they are returned in makeCredential
responses too. See linked bug for details.

Fixes w3c#1370
@nadalin nadalin added this to the L2-WD-03 milestone Mar 18, 2020
@equalsJeffH equalsJeffH self-requested a review March 18, 2020 19:13
@nadalin nadalin requested a review from akshayku March 18, 2020 19:13
Copy link
Contributor

@equalsJeffH equalsJeffH left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM (one minor suggestion), thx @agl :)

index.bs Outdated Show resolved Hide resolved
@agl
Copy link
Contributor Author

agl commented Mar 25, 2020

(Good to merge once I take care of suggested change.)

index.bs Outdated Show resolved Hide resolved
Including Jeff and Emil's comments.t

Co-Authored-By: =JeffH <jdhodges@google.com>
Co-Authored-By: Emil Lundberg <emil@emlun.se>
@equalsJeffH equalsJeffH merged commit 50679f5 into w3c:master Apr 2, 2020
@agl agl deleted the sigcount branch April 2, 2020 22:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Signature counters exist in makeCredential too
5 participants