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

Do not require setup of PID credentials for development environments #5327

Closed
djbrooke opened this issue Nov 19, 2018 · 4 comments
Closed

Do not require setup of PID credentials for development environments #5327

djbrooke opened this issue Nov 19, 2018 · 4 comments
Assignees

Comments

@djbrooke
Copy link
Contributor

Developers wanting to hack on Dataverse or evaluate Dataverse currently need to set up a test account for PID minting. This makes it harder to hack on Dataverse, so consider removing this dependency from the process.

Production installations and those wanting to test the full publish flow would still need some way to mint PIDs.

Other ideas welcome, but the goal is reducing what's needed to work on and/or run the code in a development/test environment.

@matthew-a-dunlap
Copy link
Contributor

matthew-a-dunlap commented Nov 19, 2018

I just spun up a related issue on not requiring an email server for development #5328

@poikilotherm
Copy link
Contributor

poikilotherm commented Nov 19, 2018

This might be related to #4106, #5082 and others. @4tikhonov and I discussed plans about implementing an external service for PID minting that is used via some REST API or similar. That would offload maintenance of PID providers from the main project and it might offer a solution to have a dummy/local only PID minter for demo/dev/... purposes available.

Maybe we can ask @4tikhonov during the next Community Call how far they got at DataverseEU with this? (I had no time to look into this yet, because of priority goal #5292 etc.)

@djbrooke
Copy link
Contributor Author

djbrooke commented Nov 28, 2018

We'll use internal mock implementation of a PID provider. It should be obvious that it's a test PID through the UI.

@matthew-a-dunlap
Copy link
Contributor

matthew-a-dunlap commented Dec 18, 2018

For what its worth, this is the current error on my dev machine. Forgive my randomly spammed dataset name:

screen shot 2018-12-18 at 11 55 47 am

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants