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
#2612 added new review setting for acceptance warning. #2645
Conversation
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.
Just a Q: Does this really close #2612?
There is one other task on that issue:
Inform person running pre-pub checklist that metadata is going to be redeposited with Crossref to handle updates
Hey @mauromsl, just seeing your question now. Andy and I discussed this after he made the PR and we decided I am supposed to add something to it, but I need a function from another branch that hasn't been merged yet, #2631 . So once you or @ajrbyers review and merge that branch, I can work on this. IMHO we should move it out of 1.4.1 to avoid delays. |
Ah yes it can @ajrbyers. Will start working on it today |
Should be ready for re-review now @mauromsl |
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.
This is very neat. A few changes/comments requested in-line
@mauromsl This one is ready to be merged too, I believe |
Closes #2612