Update Confirm and Submit page to match version 5 mockups.#55
Open
DavidLFielding wants to merge 20 commits into
Open
Update Confirm and Submit page to match version 5 mockups.#55DavidLFielding wants to merge 20 commits into
DavidLFielding wants to merge 20 commits into
Conversation
…to a friendly HTML page) when the bucket has no PDF (edgecase); compilation is curently broken so self-heal when PDF copied into place (review): when the bucket has a PDF but submission.preview is None or stale, fire ConfirmSourceProcessed and ConfirmPreview atomically so the second event sees the in-memory state from the first. The legacy DB doesn't persist submission.preview, so a reload between events would lose it. INFO logging for each branch. [SUBMISSION-122] David
…harden /preview.pdf: catch NotFound from the controller and 302 to /preview_not_available (non-.pdf URL so the browser renders HTML instead of invoking the PDF viewer on a .pdf URL that returned HTML), and make set_etag/Content-Length best-effort so missing blob metadata doesn't crash the response. [SUBMISSION-122] David
…exists in bucket) and pass to template so an absent PDF can never produce an enabled Submit button. Skip FinalizeSubmission unless the form action is next -- the nav bar's Go Back and Save & Exit buttons also POST this form, which would accidentally submit when the proceed checkbox happens to be ticked. INFO logging on render. [SUBMISSION-122] David
…it gated on preview_ready (preview viewed AND PDF still exists in bucket); banner branches three ways (no PDF / unviewed PDF / ready); sidebar adds Download Submission Agreement link and editing-after-submitting copy; new shortcuts row with Preview PDF (tracking preview state), Edit Metadata, Edit Files, Edit License; JS state machine reloads the page on tab return after a PDF view; amber highlight on the proceed checkbox row when ticking it is the last remaining gate. [SUBMISSION-122] David
…nd Approve' -> 'Confirm and Submit'. Sets the pagetitle context var which arxiv-base renders into <title> in the page head. [SUBMISSION-122] David
…ew" -> "Confirm", title "Final preview" -> "Confirm and Submit". Class name stays so the endpoint URL is unchanged; docstring notes the rename. [SUBMISSION-122] David
…ubmit_nav and expose it via named blocks so per-page templates can customize the Submit button without rewriting the macro: submit_nav gains next_label/next_aria/next_disabled/next_id (backward-compat defaults of "Continue" / "Save and continue" / not disabled / no id preserve current behavior for every existing page), and base_edit.html wraps the top/bottom submit_nav calls in submit_nav_top and submit_nav_bottom blocks so final_preview can override and render "Submit" instead. [SUBMISSION-122] David
…rsor on disabled .submit-nav buttons because Bulma's default opacity-only disabled styling reads as active on a saturated green Submit, and add .proceed-highlight (soft amber row + bolder label + brief pulse animation) to draw the eye to the proceed checkbox when ticking it is the only remaining gate before Submit. [SUBMISSION-122] David
…directs here because the bucket has no PDF: explains the situation as unexpected (the PDF should exist by Confirm step), suggests retry/reprocess/contact-support, and offers Back-to-Confirm / Go-to-Process / Close-tab buttons. Lives at a non-.pdf URL so the browser renders HTML instead of invoking its built-in PDF viewer. [SUBMISSION-122] David
…mission Agreement link, plus the reportlab dependency it needs. Generates a personalized PDF stamped with the submission's ID, submitter, submission date, license, and agreement_id using reportlab Platypus; body text is placeholder pending canonical legal copy from arXiv. [SUBMISSION-122] David
…lob metadata on read so crc32c, size, and updated timestamps are populated, avoiding ETag/Content-Length failures and giving callers real metadata. [SUBMISSION-122] David
…greement controller commit. [SUBMISSION-122] David
…riting agreement_id to the row (via update_from_submission in models.py) but never reading it back, so every reload reset submission.agreement_id to None. Surfaced when the new Submission Agreement PDF showed "Agreement ID: (unknown)" for properly-completed submissions. One-line fix: read row.agreement_id into the domain.Submission constructor. [SUBMISSION-122] David
|
For the check box: |
Contributor
Author
Not at this time. You can't submit a submission without selecting the checkbox confirming you have reviewed the submission. We do record viewed PDF in the DB. |
… submission type warning (moved to Review Files step). [SUBMISSION-149] David
jimentwood
approved these changes
May 28, 2026
jimentwood
left a comment
jimentwood
left a comment
There was a problem hiding this comment.
I'm not reviewing code, but the screenshots and functions described look good to me.
bdc34
approved these changes
May 28, 2026
SUBMISSION-149: Updated Upload Files page per v5 mockup. Fixed notification messages. Removed unknown source format warning.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This page updates the Update Confirm and Submit page (was Preview) to match version 5 mockups. There are multiple edge case failures and bugs that have also been addressed in this PR (ex. The 'Go Back' button submits the submission when the confirmation checkbox is checked.)
The main update is to support version 5 of the Confirm and Submit mockup.
@jimentwood The Confirm and Submit page automatically detects when the PDF has been viewed without refreshing the page. When the submitter returns to the Confirm and Submit page the page is updated to reflect the PDF has been viewed. No need to refresh the page.
@SBBCornell One subtle change: after the PDF has been view the red 'Preview PDF' button becomes white and there is a lack of 'focus' for what the submitter should do next. The Submit button is not active until the submitter checks the confirm checkbox. To draw attention to this, I added a highlight to the line asking the submitter to confirm that they have reviewed the PDF.
Submit 1.5 screenshot: (legacy)
Submit 2.0 Preview Page (original)
Submit 2.0 Confirm and Submit page mockup (target)
Updates:
Submit 2.0 Updated Confirm and Submit page (Initial view - Preview of PDF Required)
Submit 2.0 Updated Confirm and Submit page (Preview complete, focus moved to confirm reviewed checkbox)
Submit 2.0 Updated Confirm and Submit page (Preview complete, confirmation complete, Submit button activated)
Submit 2.0 Updated Confirm and Submit page (edge case - internal - No PDF)
Dynamically generated customized Submission Agreement PDF:
arxiv-submission-agreement-2.pdf