Skip to content

Fixing external tools with guestbook responses - #12801

Open
stevenwinship wants to merge 2 commits into
developfrom
12760-previewers-not-working-with-guestbook-responses
Open

stevenwinship wants to merge 2 commits into
developfrom
12760-previewers-not-working-with-guestbook-responses

Conversation

@stevenwinship

@stevenwinship stevenwinship commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

What this PR does / why we need it: External tools can't download files that require/have a guestbook response

Which issue(s) this PR closes:#12760

Special notes for your reviewer:

Suggestions on how to test this: Create a dataset with multiple files to preview. Add a guestbook to the dataset and publish. Test previewing the files and download from the external tool. Test files with restricted access.
Download timeout was increased from 1 minute to 5 minutes. Verify the user has more time to download.

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?: included

Additional documentation:

@stevenwinship stevenwinship self-assigned this Oct 6, 2026
@stevenwinship stevenwinship added Size: 30 A percentage of a sprint. 21 hours. (formerly size:33) Type: Bug a defect labels Oct 6, 2026
@stevenwinship stevenwinship added this to the 6.13 milestone Oct 6, 2026
@coveralls

coveralls commented Oct 6, 2026 •

Copy link
Copy Markdown

Coverage Status

Coverage is 25.553% — 12760-previewers-not-working-with-guestbook-responses into develop. No base build found for develop.

@github-actions

This comment has been minimized.

@github-actions

github-actions Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

Test Results

407 tests   392 ✅  32m 23s ⏱️
 55 suites   15 💤
 55 files      0 ❌

Results for commit 86741c7.

♻️ This comment has been updated with latest results.

@sonarqubecloud

sonarqubecloud Bot commented Oct 6, 2026

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
32.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@github-actions

github-actions Bot commented Oct 6, 2026

Copy link
Copy Markdown

📦 Pushed preview images as

ghcr.io/gdcc/dataverse:12760-previewers-not-working-with-guestbook-responses
ghcr.io/gdcc/configbaker:12760-previewers-not-working-with-guestbook-responses

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

@stevenwinship stevenwinship removed their assignment Oct 6, 2026
@stevenwinship stevenwinship added the FY27 Sprint 7 FY27 Sprint 7 (2026-09-23 - 2026-10-07) label Oct 6, 2026
@landreev landreev self-assigned this Oct 7, 2026
@cmbz cmbz added the FY27 Sprint 8 FY27 Sprint 8 (2026-10-07 - 2026-10-21) label Oct 7, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

FY27 Sprint 7 FY27 Sprint 7 (2026-09-23 - 2026-10-07) FY27 Sprint 8 FY27 Sprint 8 (2026-10-07 - 2026-10-21) Size: 30 A percentage of a sprint. 21 hours. (formerly size:33) Type: Bug a defect

Projects

Status: In Review 🔎

Development

Successfully merging this pull request may close these issues.

Previewers no longer working on files with required guestbooks, due to the API changes (since 6.10)

4 participants