Closed kjgarza closed 4 years ago
Check out this pull request on
Review Jupyter notebook visual diffs & provide feedback on notebooks.
Powered by ReviewNB
@mfenner I added the first version of the maDMP for the single DMP. There is a sample of the queries I am specking to be able to do to graphQL to get the queries in both notebooks. Please do take a look.
By the way is the first use of the DRAFT PR requests and the new PR template as we agreed in the Virtual Dev Meeting
@kjgarza what is it that I should review? Is it a working notebook and I should check that it is working as expected? Or is there a specific aspect you want feedback on?
@mfenner well I don't know how you review notebooks. But i guess whether we can considered closed?
specifically notebook dmp/user-story-single-dmp-connections.ipynb
is the one needs to be reviewed. Should I add Sarala in the review? I actually didn't' thought about that
Purpose
Added notebooks to address the use cases of:
Notebook 1: Show everything that is connected to a single DMP. This demonstrates the vision of a DMP as a container for a research project. Datasets (DOI) Funding (DOI) People (ORCID) Organizations (RoR)
closes: https://github.com/datacite/datacite/issues/1080
further details: https://docs.google.com/document/d/1LRP8GbyO-z1I4fjmvb4dsnsHzayJR3TxvpDcjP32iis/edit# https://docs.google.com/document/d/1LMuqAHe5y6cXPJQYT7X7vmspo0o5KhTt5Nyy_yYz5Tw/edit#
Approach
Open Questions and Pre-Merge TODOs
Learning
Types of changes
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
Reviewer, please remember our guidelines: