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

Update and rename ErinBonner_BCO_000531 to Erin_Bonner_BCO_000531 #175

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ebonner13
Copy link
Contributor

Execution domain: The selected publication does not list any specifications.

@ebonner13 ebonner13 self-assigned this Oct 16, 2019
@ebonner13 ebonner13 added this to the Final BCO milestone Oct 16, 2019
@ebonner13
Copy link
Contributor Author

The Execution Domain is not filled in because the selected publication does not list any specifications, as discussed with Janisha Patel.

@ebonner13
Copy link
Contributor Author

My BCO is ready for final review before pFDA submission.

Copy link
Collaborator

@jpat1546 jpat1546 left a comment

Choose a reason for hiding this comment

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

@ebonner13
I've reviewed your updated BCO, and made comments on what further needs to be fixed. Please make all the changes in the BCO editor, download, and update this pull request with the JSON (copy/paste- same process as before). Once all changes have been made, assign this pull request back to me and I will review again. Until then, assigning this pull request back to you.

"orcid": "",
"affiliation": "Department of Medicinal Chemistry, College of Pharmacy, Rogel Cancer Center, University of Michigan, Ann Arbor, MI 48109-2800, USA",
"contribution": [
"authoredBy"
Copy link
Collaborator

Choose a reason for hiding this comment

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

In addition to "authoredBy", each publication author should also have "derivedFrom" contribution type (See #112).
Please add.

"orcid": "",
"affiliation": "Department of Computational Medicine and Bioinformatics, University of Michigan, Ann Arbor, MI 48109-2800, USA",
"contribution": [
"authoredBy"
Copy link
Collaborator

Choose a reason for hiding this comment

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

In addition to "authoredBy", each publication author should also have "derivedFrom" contribution type (See #112).
Please add.

"orcid": "https://orcid.org/0000-0003-1553-6695",
"affiliation": "Departments of Radiation Oncology, Rogel Cancer Center, University of Michigan Medical School and Department of Environmental Health Sciences, School of Public Health, University of Michigan, Ann Arbor, MI 48109-2800, USA",
"contribution": [
"authoredBy"
Copy link
Collaborator

Choose a reason for hiding this comment

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

In addition to "authoredBy", each publication author should also have "derivedFrom" contribution type (See #112).
Please add.

"orcid": "https://orcid.org/0000-0003-3291-7131",
"affiliation": "Department of Medicinal Chemistry, College of Pharmacy, Rogel Cancer Center, University of Michigan, Ann Arbor, MI 48109-2800, USA",
"contribution": [
"authoredBy"
Copy link
Collaborator

Choose a reason for hiding this comment

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

In addition to "authoredBy", each publication author should also have "derivedFrom" contribution type (See #112).
Please add.

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

Successfully merging this pull request may close these issues.

2 participants