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

Note on reduced scope of dcat:Distribution #299

Merged
merged 7 commits into from
Jul 16, 2018
Merged

Note on reduced scope of dcat:Distribution #299

merged 7 commits into from
Jul 16, 2018

Conversation

dr-shorthair
Copy link
Contributor

  1. Add NOTE describing recommendations around the reduced scope of dcat:Distribution, pointing users to the new *Service classes
    2.fix up a couple of seeAlsos

Proposed fix for issue identified by @makxdekkers in #52

1. Add NOTE describing recommendations around the reduced scope of dc…
034ed24 
…at:Distribution, pointing users to the new *Service classes

2.fix up a couple of seeAlsos
@dr-shorthair dr-shorthair requested a review from makxdekkers July 10, 2018 02:53
@dr-shorthair
Copy link
Contributor Author

Preview at https://rawgit.com/w3c/dxwg/dcat-issue52-simon/dcat/index.html#class-distribution

@dr-shorthair dr-shorthair mentioned this pull request Jul 10, 2018
Closed
@dr-shorthair
Copy link
Contributor Author

dr-shorthair commented Jul 12, 2018 •
edited
Loading

Additional mods to complete actions 1 & 2 https://www.w3.org/2018/07/12-dxwgdcat-minutes#ActionSummary

@agbeltran @davebrowning Should be good to go now, per decision in yesterday's meeting.

@agbeltran agbeltran self-requested a review July 16, 2018 23:35
agbeltran
agbeltran approved these changes Jul 16, 2018
View reviewed changes
Copy link
Member

@agbeltran agbeltran left a comment

Choose a reason for hiding this comment

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

Approved including a few edits I pushed.

@agbeltran agbeltran merged commit 9eeca3d into gh-pages Jul 16, 2018
@agbeltran agbeltran deleted the dcat-issue52-simon branch July 16, 2018 23:35
@dr-shorthair dr-shorthair mentioned this pull request Feb 6, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@agbeltran agbeltran agbeltran approved these changes

@makxdekkers makxdekkers Awaiting requested review from makxdekkers

Assignees

@pwin pwin

@agbeltran agbeltran

@davebrowning davebrowning

Labels
dcat:DataService dcat:Distribution dcat
Projects
None yet
Milestone
DCAT Second Public Working Draft
Development

Successfully merging this pull request may close these issues.
4 participants
@dr-shorthair @agbeltran @pwin @davebrowning