-
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
205 - Integration dataset user permissions #218
205 - Integration dataset user permissions #218
Conversation
…tory integration test
…into feature/205-integration-dataset-user-permissions
…mission integration
@MellyGray Please, can you resolve the conflicts? |
…into feature/205-integration-dataset-user-permissions
@GPortas I fixed the merge conflicts |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm having the following error while building the docker image, @MellyGray:
@GPortas I followed the following steps and the docker image finished successfully:
I would clear npm cache and try again, I checked and that version exists in the npm registry so it shouldn't send a not found |
…-user-permissions 205 - Integration dataset user permissions
What this PR does / why we need it:
This PR adds the integration of the SPA with the js-dataverse implementation to get the dataset user permissions
Which issue(s) this PR closes:
Special notes for your reviewer:
Suggestions on how to test this:
Step 1: Run the development environment
npm I
cd packages/design-system && npm run build
cd ../../
.env
file such as the .env.example, with theVITE_DATAVERSE_BACKEND_URL=http://localhost:8000
variablecd dev-env
./run-env.sh unstable
Step 2: Test Dataset View mode with the implemented changes for the getFilesTotalDownloadSize
?persistentId=doi:10.5072/FK2/LHGRHP&version=DRAFT
Does this PR introduce a user interface change? If mockups are available, please link/include them here:
No
Is there a release notes update needed for this change?:
No
Additional documentation: