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

Harvest attribution roles #3149

Draft
wants to merge 16 commits into
base: master
Choose a base branch
from
Draft

Harvest attribution roles #3149

wants to merge 16 commits into from

Conversation

maudetes
Copy link
Contributor

@maudetes maudetes commented Sep 18, 2024 •
edited by magopian
Loading

Closes datagouv/data.gouv.fr#1588

  • contact_points is a now list of contacts
    • migration included
  • a contact point has a role (contact, publisher, etc.)
  • harvest dct:publisher, dct:creators, dct:contributors.
  • only expose one dct:publisher, so if there's a contact point with the role publisher, the owner should be prov:qualifiedAttribution instead
  • expose dct:publisher, dct:creators, dct:contributors in RDF (with correct properties depending on role)
  • do we want a dedicated attribution field instead of contact point? -> not for now
  • make udata-front resilient with contact points list
  • update and add tests
  • play around to add a suggest API on these contacts
  • add a changelog

@maudetes
Harvest responsible roles in extras
e8fb579 
Support FOAF.name for dct:publisher, dct:creators, dct:contributors
@maudetes
Play around to create a responsible party suggest API
a0e8d74
@maudetes maudetes marked this pull request as draft September 18, 2024 13:47
maudetes added a commit to datagouv/udata-front that referenced this pull request Sep 18, 2024
@maudetes
Add publishers, creators and contributors on dataset page
faf2656 
This is based on opendatateam/udata#3149
@maudetes maudetes mentioned this pull request Sep 18, 2024
Draft
@maudetes maudetes changed the title Harvest responsible roles in extras Harvest attribution roles Dec 10, 2024
@magopian magopian force-pushed the feat/add-roles-in-harvest-extras branch from 3d8fe90 to 609150c Compare December 16, 2024 15:36
@magopian magopian mentioned this pull request Dec 16, 2024
Open
@magopian magopian force-pushed the feat/add-roles-in-harvest-extras branch from 609150c to e200393 Compare December 16, 2024 15:39
@magopian
Make the fixture commands less brittle
a156612
@magopian
Don't nest the dataset in its resources, discussions, reuses and data…
b66c890 
…services when generating the fixtures

Also auto-set a role on the contact point if there is none
- if email or contact_form present: `contact`
- else `creator`
@magopian
Update the migration to add a default contact role for each existin…
e031796 
…g contact point
@magopian
If a contact point has role publisher, then the owner has role `PRO…
8bc47ff 
…V:qualified_attribution`
@magopian
Test contact_points_to_rdf for roles
367b540
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ThibaudDauce ThibaudDauce Awaiting requested review from ThibaudDauce

@bolinocroustibat bolinocroustibat Awaiting requested review from bolinocroustibat

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Gestion des multi producteurs
2 participants
@maudetes @magopian