search

arXiv / arxiv-submission-ui

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

ARXIVNG-2885: Implement TeX Produced display components in Submission UI #181

Closed DavidLFielding closed 4 years ago

DavidLFielding commented 4 years ago

This task [ ARXIVNG-2885 ] adds support for TeX-Produced check at the submission UI to provide feedback to users who attempt to submit a TeX generated PDF.

There are two PRs for this work, the other implements the actual check in the arxiv-submission-core repository. This PR implements changes to the user interface to display a message when we detect a TeX produced single-file submission.

During testing I also ran into internal errors from the Compilation service that are not displayed in an informative way. I've added a flash message and steps to take message for this error condition instead of basically displaying nothing.

I've also made some updates to support the latest preview/auth changes.

This PR require a version of the arxiv-submission-core package (0.7.2rc14) that supports the TeX-Produced check. The Pipfile and Pipfile.lock have been updated with this package.

I was unable to attach screenshots to this PR. I tried a few different formats including PNG, JPEG. I've added the following screenshots to the original ticket:

New TeX Produced warning
Legacy TeX Produced message
Compilation service failure message

Two possible test suggestions:

To generate TeX Produced message simply upload a PDF that has been generated from TeX source.

To generate Compiler service error set your convert image to something that doesn't exist and try to submit a TeX source package. This error is secondary to main purpose of ticket.

Shamsi: You will be most interested in the warning messages in the UI. I basically copied the original TeX Produced error from the legacy system as a starting point. I also created a basic message for the compilation service failure. Let me know if you have any suggestions here. I've also observed that the vertical spacing between the "Go back to Upload Files" and "Reprocess" buttons seems excessive (expands) onTeX-produced error pages (for both PDF and Postscript).

SBBCornell commented 4 years ago

I tested by downloading an existing PDF from the live arxiv site and uploading just that file during submission. I expected to get errors - because its PDF generated from TeX - but I got a success message instead. Should I be testing in a different way?

DavidLFielding commented 4 years ago

The tests that never should have ever succeeded now succeed!! Yes!

I still need to clean up the conflicts. Working on that now.

DavidLFielding commented 4 years ago

Conflicts are taken care of.