Skip to content

inside out Croissant for review datasets#12221

Draft
pdurbin wants to merge 1 commit into12014-croissant-1.1from
12124-inside-out-croissant
Draft

inside out Croissant for review datasets#12221
pdurbin wants to merge 1 commit into12014-croissant-1.1from
12124-inside-out-croissant

Conversation

@pdurbin
Copy link
Member

@pdurbin pdurbin commented Mar 16, 2026

What this PR does / why we need it:

For review datasets, we are trying to represent the item reviewed in the Croissant in the head of the HTML.

This differs from all other dataset types in which we represent the dataset itself.

Which issue(s) this PR closes:

Special notes for your reviewer:

The most important file to review is this one: src/test/resources/croissant/review/expected/review-croissantSlim.json

It shows how the Croissant will look for review datasets in the "head" for the HTML.

Suggestions on how to test this:

Does this PR introduce a user interface change? If mockups are available, please link/include them here:

Is there a release notes update needed for this change?:

Additional documentation:

@github-actions github-actions bot added Croissant Croissant and Kaggle related work Project: Trusted Data labels Mar 16, 2026
@pdurbin pdurbin changed the title implemente inside out croissant for review datasets #12124 inside out Croissant for review datasets Mar 16, 2026
@pdurbin pdurbin moved this to In Review 🔎 in IQSS Dataverse Project Mar 16, 2026
@pdurbin pdurbin requested a review from cmbz March 16, 2026 21:02
@coveralls
Copy link

Coverage Status

coverage: 24.905% (+0.005%) from 24.9%
when pulling cc911a3 on 12124-inside-out-croissant
into 97ff858 on 12014-croissant-1.1.

@github-actions
Copy link

📦 Pushed preview images as

ghcr.io/gdcc/dataverse:12124-inside-out-croissant
ghcr.io/gdcc/configbaker:12124-inside-out-croissant

🚢 See on GHCR. Use by referencing with full name as printed above, mind the registry name.

@pdurbin pdurbin moved this from In Review 🔎 to In Progress 💻 in IQSS Dataverse Project Mar 19, 2026
@pdurbin pdurbin assigned pdurbin and unassigned cmbz Mar 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Croissant Croissant and Kaggle related work Project: Trusted Data

Projects

Status: In Progress 💻

Development

Successfully merging this pull request may close these issues.

3 participants