-
Notifications
You must be signed in to change notification settings - Fork 16
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
FIx dataset loading logic for Deaccessioned Dataset #273
Comments
@ekraffmiller I attempted to replicate the issue and encountered an error with the 'get dataset deaccessioned' endpoint when the user is not authenticated. Therefore, I'm assuming you discovered this bug using a different image of Dataverse where the initial issue is resolved. I tried running another image using I believe we need to first merge the above PR, as it currently seems impossible to open a deaccessioned dataset in the frontend if the user is unauthenticated. Also, could you provide some steps on how to reproduce the bug regarding the 'getDatasetFilesTotalDownloadSize'? I'm going to add the Waiting tag for the moment |
Deprioritized and moved back to the SPA backlog. This issue is not as important as others for the Beta Release and also requires further discussion. |
@MellyGray I created a PR with steps to reproduce, I hope that makes sense, let me know if you have any questions or problems reproducing it |
When a dataset is deaccessioned, it still should be visible to unauthenticated users, but only the citation and deaccession reason are viewable.
For example in JSF:
There is a bug in the SPA where getDatasetFilesTotalDownloadSize() is called for a deaccessioned dataset and an unauthenticated user. This API call returns an error because the user doesn't have permission to view the download sizes for a deaccessioned dataset. Because of this error, the SPA redirects to a published version of the dataset.
We need to update the logic of the SPA so that getDatasetFilesTotalDownloadSize() and other file related API's are only called if the user has permission to view the files. For example, if the dataset is deaccessioned, but the user has edit permissions on the dataset.
The text was updated successfully, but these errors were encountered: