Skip to content

PrePublish disclaimer for submit for review#12373

Merged
landreev merged 6 commits into
developfrom
12325-submit-for-review-message
May 5, 2026
Merged

PrePublish disclaimer for submit for review#12373
landreev merged 6 commits into
developfrom
12325-submit-for-review-message

Conversation

@stevenwinship

@stevenwinship stevenwinship commented May 1, 2026

Copy link
Copy Markdown
Contributor

What this PR does / why we need it: Need a message explains Submit for Review so users are aware of the workflow.

Which issue(s) this PR closes:#12325

Special notes for your reviewer:

Suggestions on how to test this: Sample settings values
curl -sS -X PUT -d 'I agree to the following:<br/>1. My submission has been fully anonymized (required for all human subjects'\'' datasets).<br/>2. My submission does not violate any known copyright laws.<br/>3. I understand that I am liable for any and all violations of the Harvard Repository <a href=https://support.dataverse.harvard.edu/harvard-dataverse-general-terms-use>Terms of Use.</a>' http://localhost:8080/api/admin/settings/:SubmitForReviewDatasetDisclaimerText

curl -sS -X PUT -d 'This dataset is being submitted for review and publishing.<br/><span style="color: red;">If PUBLISHING is not your intended workflow</span>, do not submit this for review and contact <a href="https://support.dataverse.harvard.edu/"> support.</a>' http://localhost:8080/api/admin/settings/:DatasetSubmitForReviewPopupCustomText

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

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

Additional documentation:

@coveralls

coveralls commented May 1, 2026

Copy link
Copy Markdown

Coverage Status

coverage: 24.956% (-0.002%) from 24.958% — 12325-submit-for-review-message into develop

@github-actions

This comment has been minimized.

@stevenwinship stevenwinship added the FY26 Sprint 22 FY26 Sprint 22 (2026-04-22 - 2026-05-06) label May 1, 2026
@stevenwinship stevenwinship added this to the 6.11 milestone May 1, 2026
@github-actions

This comment has been minimized.

2 similar comments
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@stevenwinship
stevenwinship force-pushed the 12325-submit-for-review-message branch from 7bc5391 to a46655b Compare May 2, 2026 14:09
@stevenwinship stevenwinship moved this from In Progress 💻 to Ready for Review ⏩ in IQSS Dataverse Project May 2, 2026
@stevenwinship stevenwinship removed their assignment May 2, 2026
@github-actions

This comment has been minimized.

@rtreacy rtreacy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good

@github-project-automation github-project-automation Bot moved this from Ready for Review ⏩ to Ready for QA ⏩ in IQSS Dataverse Project May 4, 2026
@landreev landreev moved this from Ready for QA ⏩ to QA ✅ in IQSS Dataverse Project May 4, 2026
@landreev landreev self-assigned this May 4, 2026

@landreev landreev left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Everything is working as advertised.
With the same exact settings as specified in "how to test" verbatim, the popup looks 1:1 as in the screenshot.
@stevenwinship
Just curious, where did the words "pre-publish" (as in, PrePublishDatasetDisclaimerText) come from? - instead of just referring to it as SubmitForRevieweverywhere. Is that how curation prefer to talk about this process?

"Pre publish" has been used in the context of workflows. But it means something else there.
Ready to merge generally.

@landreev

landreev commented May 4, 2026

Copy link
Copy Markdown
Contributor

Just curious, where did the words "pre-publish" ... come from

@stevenwinship n/m, I see it in the discussion in the issue (#12325).

@landreev

landreev commented May 4, 2026

Copy link
Copy Markdown
Contributor

... although I am still a little confused. Will ask a quick question in the issue.

Just curious, where did the words "pre-publish" ... come from

@stevenwinship n/m, I see it in the discussion in the issue (#12325).

@stevenwinship
stevenwinship force-pushed the 12325-submit-for-review-message branch from 20a372b to 96ebb7a Compare May 5, 2026 16:07
@github-actions

This comment has been minimized.

1 similar comment
@github-actions

This comment has been minimized.

@landreev

landreev commented May 5, 2026

Copy link
Copy Markdown
Contributor

It looks great, IMO and is still working as advertised.
I pinged @scolapasta and @pdurbin on slack just to give them an extra chance to stop me from merging the pr. Otherwise will merge shortly.

@github-actions

github-actions Bot commented May 5, 2026

Copy link
Copy Markdown

📦 Pushed preview images as

ghcr.io/gdcc/dataverse:12325-submit-for-review-message
ghcr.io/gdcc/configbaker:12325-submit-for-review-message

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

@landreev

landreev commented May 5, 2026

Copy link
Copy Markdown
Contributor

Heard back, merging.

@landreev
landreev merged commit a19d03b into develop May 5, 2026
26 of 27 checks passed
@github-project-automation github-project-automation Bot moved this from QA ✅ to Merged 🚀 in IQSS Dataverse Project May 5, 2026
@stevenwinship
stevenwinship deleted the 12325-submit-for-review-message branch May 5, 2026 21:12
@jp-tosca jp-tosca moved this from Merged 🚀 to Done 🧹 in IQSS Dataverse Project May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done 🧹

Development

Successfully merging this pull request may close these issues.

Submit for review: When a user selects this option, a message should indicate the dataset will be reviewed and published

5 participants