Skip to content

feat: replace private by published_at for Dataset#3566

Open
ThibaudDauce wants to merge 6 commits intomainfrom
replace_private_by_published_at
Open

feat: replace private by published_at for Dataset#3566
ThibaudDauce wants to merge 6 commits intomainfrom
replace_private_by_published_at

Conversation

@ThibaudDauce
Copy link
Contributor

@ThibaudDauce ThibaudDauce commented Dec 10, 2025

Fix etalab/transport-site#4240

Need to be done for Reuse and Dataservice too in other PRs. cdata is working without any change, but could be updated to use and send the new published_at field instead of private.

@ThibaudDauce ThibaudDauce force-pushed the replace_private_by_published_at branch from 88f48c5 to 1f4df7f Compare December 10, 2025 11:42
@ThibaudDauce ThibaudDauce force-pushed the replace_private_by_published_at branch from 9f3a372 to 7b17adc Compare December 10, 2025 11:51
@streino
Copy link
Contributor

streino commented Dec 12, 2025

How (if) will this be articulated with harvested records? ISO-19139 at least has a "publication date" for the metadata (not processed by udata at this point). I could see dataset.published_at being the target for this metadata date if we're ever going to map it, with consequences like:

  • dataset.published_at could possibly be anterior to dataset.created_at.
  • (un)publication on the source catalog would control publication on udata.
  • ...?

Not saying this needs to be considered at this point nor whether that potential tie is good or bad. Just raising the question, and I can possibly help dig other potential consequences if needed.

ThibaudDauce added a commit that referenced this pull request Dec 18, 2025
> 👋 je vois que l'API ne renvoie les activités d'un objet private
que pour les super-admin
https://github.com/opendatateam/udata/blob/5313356a9ad9c94bbc1cb8ae7615cd4f35debd82/udata/core/activity/api.py#L112-L117
>
> Est-ce que ce serait envisageable d'introduire un check de type
visible_by_user plutôt ?

:warning: need fixing if merged before or after
#3566
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.

NewDatagouvDatasetsJob : gérer les JDDs publiés en privé initialement

2 participants