Skip to content

Add initial proposal to the document - #29

Merged
jogu merged 3 commits into
mainfrom
issue-#1
Aug 16, 2026
Merged

jogu merged 3 commits into
mainfrom
issue-#1

Conversation

@martijnharing

Copy link
Copy Markdown
Collaborator

Add initial proposal to the document

Add initial proposal to the document
@github-actions

github-actions Bot commented Jul 31, 2026

Copy link
Copy Markdown

📄 Editor's Copy preview

https://openid.github.io/dchp/PREVIEW-DO-NOT-USE/pr-29/digital-credentials-harmonized-presentation-editors-copy.html

Preview of 6f2aa3d; updated on every push. For review only — the official specifications are published at https://openid.net/specs/

github-actions Bot pushed a commit that referenced this pull request Jul 31, 2026
Add and update links to consensus issues
github-actions Bot pushed a commit that referenced this pull request Jul 31, 2026

@jogu jogu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I support merging this as a starting point. As discussed on the call we have issues for any changes needed to the various sections and should merge this PR as-is unless there's any substantial objections.

@awoie

awoie commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

This PR contains the initial document adopted by WG consensus on the 31 July call (see #1), with the ZKP section removed and the per-section consensus notes added, as agreed there.

Our working procedures require any non-editorial PR to stay open for two weeks, so this can't merge before 14 August even though the decision is already made. Two things follow, as we discussed on the call.

Please don't raise substantive comments on this PR. Instead, carry on editing as normal: open PRs on top of this one during the two weeks, and put substantive discussion in the per-section issues (#24#28) or in new issues. Nothing needs to pause, the work just lands on top rather than in here.

What is in scope for this PR is whether it faithfully reproduces the adopted contribution, i.e., correct text, ZKP section removed, section notes right. That's what I'd ask reviewers to check.

On which: we need three approvals and currently have two (thanks @jogu). Two more would be welcome. It's a fidelity check rather than a content review, so it shouldn't take long.

Comment thread draft/digital-credentials-harmonized-presentation.md Outdated
Comment thread draft/digital-credentials-harmonized-presentation.md Outdated
Co-authored-by: Christian Bormann <chris.bormann@gmx.de>
github-actions Bot pushed a commit that referenced this pull request Aug 3, 2026

@andrewhughes3000 andrewhughes3000 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approve to merge

@jogu

jogu commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

This PR now meets all the criteria to merge as per our working group rules, namely:

  1. PR has been open 2 weeks
  2. No normative changes to the PR in the last 2 weeks
  3. The issue or the PR has been discussed in at least 1 WG meeting
  4. No unresolved comments
  5. 3 PR approvals

Merging - thanks everyone!

@jogu
jogu merged commit 549062f into main Aug 16, 2026
6 checks passed
github-actions Bot pushed a commit that referenced this pull request Aug 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants