Closed erickpeirson closed 5 years ago
Thanks! Let's keep this PR open and I'll look at those issues. I have a couple of other fixes to push, and will then ask for rereview
On Wed, Mar 6, 2019, 10:53 Liz Woods notifications@github.com wrote:
@eawoods commented on this pull request.
Looks good overall. There might be some new errors introduced in the process, particularly with routes? During testing, I got a PDF submission to process all the way through. Yay!
- TeX submission (that should have been valid) got hung and didn't finish, but I was allowed to continue through to metadata entry. (expected behavior?)
- "Save and Exit" and "Go Back" do nothing on optional metadata and preview pages
- Deleting a submission through any of the buttons leads to 404
Would you rather log these and move on to the next stage, or fix anything on this PR?
In submit/templates/submit/file_process.html https://github.com/arXiv/arxiv-submission-ui/pull/61#discussion_r261773470 :
@@ -165,6 +167,13 @@
Messages with no particular status
{% endif %}
This fixes a few issues that cropped up during the last review.
It also does a lot of DRYing-out of the
routes.ui
module.And I made other little tweaks and fixes that came up along the way.
I would not recommend doing a line-by-line review of code, as the diff is pretty huge. Instead, I'd suggest using the same approach as for https://github.com/arXiv/arxiv-submission-ui/pull/59
You may need to clear out any previous containers/volumes, and will definitely need to rebuild the submission-ui container.