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.

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

explain how to switch to (working) betatest NcML previewer #11311

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

pdurbin
Copy link
Member

@pdurbin pdurbin commented Mar 5, 2025

What this PR does / why we need it:

v1.4 of the NcML previewer doesn't work. It should be deleted. People should switch to the "betatest" version until v1.5 has been released.

Which issue(s) this PR closes:

@github-actions github-actions bot added NIH CAFE Issues related to and/or funded by the NIH CAFE project Size: 3 A percentage of a sprint. 2.1 hours. Type: Bug a defect labels Mar 5, 2025
@@ -0,0 +1,18 @@
## NcML Previewer Bug Fix

[Dataverse Previewers](https://github.com/gdcc/dataverse-previewers) v1.4 contains a bug in the NcML previewer that prevents it from working. (See #11252 for screenshots.)
Copy link
Member

Choose a reason for hiding this comment

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

It only doesn't work with signedUrls. Should still be fine with an API key.

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks, fixed in bce2d31

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NIH CAFE Issues related to and/or funded by the NIH CAFE project Size: 3 A percentage of a sprint. 2.1 hours. Type: Bug a defect
Projects
Status: Ready for QA ⏩
Development

Successfully merging this pull request may close these issues.

NcML previewer not working
2 participants