-
Notifications
You must be signed in to change notification settings - Fork 2
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
initial implementation of publication to Zenodo #107
Conversation
Use Prettier to format tables
Zenodo does not seem to accept JSON with control character 127
Sorry it's not ready for review yet |
This is ready I think. There is nothing left specific to the ELN. I put options to allow for custom things |
}); | ||
|
||
exports.createEntry = composeWithError(async (ctx) => { | ||
const { entryId } = ctx.query; |
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.
readme says entry id is in the body
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.
fixed readme
attachments to add to the Zenodo entry. It can also return an object if a single | ||
attachment is to be added | ||
|
||
```js |
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.
Is this the default or just an example?
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.
Example. There is no default.
No description provided.