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.

Sign up for GitHub

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

Jump to bottom

Adding some explanation about metadata integrity and authenticity #1542

Merged
merged 6 commits into from
Nov 21, 2022
Merged

Adding some explanation about metadata integrity and authenticity #1542

merged 6 commits into from
Nov 21, 2022

Conversation

riccardoAlbertoni
Copy link
Contributor

@riccardoAlbertoni riccardoAlbertoni added this to the DCAT3 CR milestone Nov 8, 2022
Annette Greiner and others added 2 commits November 8, 2022 13:14
Clarifying the current approach to best practices around checksums. I…
628cd65 
…f we decide to offer a separate property, such as a url pointing to a method for calculating the checksum, this will need to change.
@riccardoAlbertoni
adding a sentece about verifiable credentials
ccce7e2
@riccardoAlbertoni
Copy link
Contributor Author

Thanks, @agreiner, for complementing the proposal.

I have added a draft link sentence to verifiable credentials.
Please, @pchampin, check if that sentence mirrors your suggestions.

andrea-perego
andrea-perego reviewed Nov 10, 2022
View reviewed changes
Copy link
Contributor

@andrea-perego andrea-perego left a comment

Choose a reason for hiding this comment

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

Some suggested editorial fixes.

dcat/index.html Outdated Show resolved Hide resolved
@riccardoAlbertoni
Copy link
Contributor Author

Some suggested editorial fixes.

Thanks a lot, @andrea-perego.
I have accepted your suggestions and added a new line to the change history in the document.

@riccardoAlbertoni riccardoAlbertoni marked this pull request as ready for review November 15, 2022 22:20
@davebrowning davebrowning self-requested a review November 15, 2022 22:21
@riccardoAlbertoni riccardoAlbertoni merged commit b65fc7f into gh-pages Nov 21, 2022
@riccardoAlbertoni riccardoAlbertoni deleted the DCAT-issue-1526-bis branch November 21, 2022 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pchampin pchampin pchampin approved these changes

@davebrowning davebrowning davebrowning approved these changes

@andrea-perego andrea-perego Awaiting requested review from andrea-perego

@agbeltran agbeltran Awaiting requested review from agbeltran

@dr-shorthair dr-shorthair Awaiting requested review from dr-shorthair

@pwin pwin Awaiting requested review from pwin

Assignees
No one assigned
Labels
dcat
Projects
None yet
Milestone
DCAT 3 CR
Development

Successfully merging this pull request may close these issues.
4 participants
@riccardoAlbertoni @pchampin @andrea-perego @davebrowning