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

DCAT updating change history #1476

Merged
merged 11 commits into from Mar 22, 2022
Merged

Conversation

riccardoAlbertoni
Copy link
Contributor

@riccardoAlbertoni riccardoAlbertoni commented Mar 16, 2022

This PR contributes to #1471

Updated the history adding items for issues #1472 and #1335.

I wouldn't add to the change history details related to issues #1429, #1440, #1431, #1455, would you?

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.

Proposed changes:

  • Refer to the updated class diagram with the relevant cross-reference
  • Editorial fix

dcat/index.html Outdated Show resolved Hide resolved
dcat/index.html Outdated Show resolved Hide resolved
DCAT Sprint: Editorial revisions automation moved this from In progress to Review in progress Mar 16, 2022
@andrea-perego
Copy link
Contributor

@riccardoAlbertoni said:

This PR contributes to #1471

Updated the history adding items for issues #1472 and #1335.

I wouldn't add to the change history details related to issues #1429, #1440, #1431, #1455, would you?

Yes, maybe it is not necessary.

BTW, the changelog already includes a reference to #1440.

riccardoAlbertoni and others added 2 commits March 16, 2022 22:35
Co-authored-by: Andrea Perego <andrea-perego@users.noreply.github.com>
Co-authored-by: Andrea Perego <andrea-perego@users.noreply.github.com>
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.

Editorial fix - sorry, I missed it.

dcat/index.html Outdated Show resolved Hide resolved
riccardoAlbertoni and others added 2 commits March 16, 2022 23:59
Co-authored-by: Andrea Perego <andrea-perego@users.noreply.github.com>
DCAT Sprint: Editorial revisions automation moved this from Review in progress to Reviewer approved Mar 22, 2022
@riccardoAlbertoni riccardoAlbertoni merged commit e0c9a48 into gh-pages Mar 22, 2022
DCAT Sprint: Editorial revisions automation moved this from Reviewer approved to Done Mar 22, 2022
@riccardoAlbertoni riccardoAlbertoni deleted the DCAT_updating_change_history branch March 22, 2022 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

3 participants