Skip to content

Conversation

@erickpeirson
Copy link
Contributor

@erickpeirson erickpeirson commented Apr 22, 2019

In support of arXiv/arxiv-submission-ui#80, this PR focuses on two major components of the NG submission system project:

  1. The arxiv.submission package. This package provides an event-based Python API for mutating submissions, and is the only mechanism for writing submission data to the submission database in arXiv-NG. This package is used by submission interfaces, and by the submission agent (which is also in this repository).

  2. The submission agent. This is a Kinesis consumer that orchestrates backend processes based on rules triggered by submission events.

Things to look at:

Comments are welcome that suggest specific improvements, and that focus on:

  • Possible bugs in the implementation;
  • Areas where documentation or code are unclear.

Note: I tagged folks from the admin team, since this may be useful/interesting background.

Co-Authored-By: Martin Lessmeister <[email protected]>
metadata = [
('title', '优秀的称号'),
('abstract', "当我有一天正在上学的时候当我有一天正在上学的时候"),
('comments', '5页2龟鸠'),
Copy link
Contributor

Choose a reason for hiding this comment

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

乌龟一路下来

Copy link
Contributor Author

Choose a reason for hiding this comment

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

🐢

Copy link
Contributor

@mhl10 mhl10 left a comment

Choose a reason for hiding this comment

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

Tremendous work! There's a ton of complexity here that my comments and review only scratch the surface of, but I am strongly reassured by the overall design, accompanying tests, and documentation. I look forward to seeing how this will evolve to support even more diverse submission, moderation and administrative workflows.

@erickpeirson erickpeirson merged commit e3e5f40 into master Jul 23, 2019
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

Successfully merging this pull request may close these issues.

7 participants