arXiv / arxiv-submission-ui

User interface of NG submit system.
MIT License
2 stars 6 forks source link

ARXIVNG-899 stubs for license, policy, classification, secondary classification #5

Closed JaimieMurdock closed 6 years ago

JaimieMurdock commented 6 years ago

The purpose of this ticket is to get stubs in place for several controllers and allow some immediate discussion of high level architectural decisions before finalizing.

What I see that can be improved:

For scoping, this PR is meant just to get the skeletons in for the license, policy, classification, and crosslist so that we can unblock ARXIVNG-874, 875, 876, and 877. It's meant to spur discussion, but not to necessitate the above changes - we just want to get to a working prototype and then have things we can refactor as we create tests next week.

coveralls commented 6 years ago

Pull Request Test Coverage Report for Build 50


Changes Missing Coverage Covered Lines Changed/Added Lines %
submit/routes/ui.py 0 3 0.0%
submit/controllers/init.py 0 4 0.0%
submit/controllers/authorship.py 0 5 0.0%
submit/controllers/verify_user.py 0 6 0.0%
submit/controllers/license.py 0 20 0.0%
submit/controllers/policy.py 0 20 0.0%
submit/controllers/util.py 0 27 0.0%
submit/controllers/classification.py 0 29 0.0%
<!-- Total: 0 114 0.0% -->
Files with Coverage Reduction New Missed Lines %
submit/controllers/authorship.py 1 0.0%
submit/controllers/verify_user.py 2 0.0%
<!-- Total: 3 -->
Totals Coverage Status
Change from base Build 33: -0.9%
Covered Lines: 4
Relevant Lines: 261

💛 - Coveralls
erickpeirson commented 6 years ago

I went ahead and did this so that we can move forward on other tickets. Thanks all!