From cba1d82f2f772d48c639fbad52f5b0ed45d9c5e9 Mon Sep 17 00:00:00 2001 From: kyokukou Date: Wed, 9 Sep 2026 06:56:25 -0700 Subject: [PATCH 1/2] to-do has a dash --- app/daily_update/templates/report_body.py | 2 +- tests/daily_update/test_process.py | 2 +- tests/daily_update/test_report.py | 4 ++-- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/app/daily_update/templates/report_body.py b/app/daily_update/templates/report_body.py index 236a627..7ce9210 100644 --- a/app/daily_update/templates/report_body.py +++ b/app/daily_update/templates/report_body.py @@ -6,7 +6,7 @@ from app.daily_update import announce MOD_TODO_URL = "https://check.arxiv.org/q/todo" -MOD_TODO_TITLE = "Your moderation todo queue" +MOD_TODO_TITLE = "Your moderation to-do queue" def announce_line() -> str: """the header line about when everything not on hold will be announced""" diff --git a/tests/daily_update/test_process.py b/tests/daily_update/test_process.py index 68350ea..724151e 100644 --- a/tests/daily_update/test_process.py +++ b/tests/daily_update/test_process.py @@ -251,7 +251,7 @@ def test_the_whole_html_digest_for_one_moderator(sends): assert sends["digest-cat@example.com"]["html_body"] == ( '

Daily moderator report for cs.AI

\n' '

If no further actions are taken, all submissions below not currently on hold will be announced at 09-03 20:00 EDT.

\n' - '

Your moderation todo queue

\n' + '

Your moderation to-do queue

\n' '

On Hold:

\n' '

07-27 15:00 EDT   cs.AI   Frank Franky   submit/210
\n' 'On Mod Hold
\n' diff --git a/tests/daily_update/test_report.py b/tests/daily_update/test_report.py index edc0ba6..d3c58cb 100644 --- a/tests/daily_update/test_report.py +++ b/tests/daily_update/test_report.py @@ -63,7 +63,7 @@ def test_header_text_is_the_whole_layout(): "\n" f"{announce_line()}\n" "\n" - "Your moderation todo queue: https://check.arxiv.org/q/todo\n" + "Your moderation to-do queue: https://check.arxiv.org/q/todo\n" ) @@ -71,7 +71,7 @@ def test_header_html_is_the_whole_layout(): assert render_header("cs.AI").html == ( "

Daily moderator report for cs.AI

\n" f"

{announce_line()}

\n" - '

Your moderation todo queue

\n' + '

Your moderation to-do queue

\n' ) From 84ca652c29218b279cc96a2b5383c3d46619d6bc Mon Sep 17 00:00:00 2001 From: kyokukou Date: Wed, 9 Sep 2026 07:43:42 -0700 Subject: [PATCH 2/2] change new header copy --- app/daily_update/templates/report_body.py | 2 +- tests/daily_update/test_process.py | 4 ++-- tests/daily_update/test_report.py | 8 ++++---- 3 files changed, 7 insertions(+), 7 deletions(-) diff --git a/app/daily_update/templates/report_body.py b/app/daily_update/templates/report_body.py index 7ce9210..ac46944 100644 --- a/app/daily_update/templates/report_body.py +++ b/app/daily_update/templates/report_body.py @@ -21,7 +21,7 @@ def announce_line() -> str: class Section(str, Enum): """the parts of the report, listed in the order they appear. the value is the heading used in the email.""" HOLD = "On Hold" - NEW = "New" + NEW = "Scheduled for announcement" CROSS = "Cross Lists" NOTHING_TO_REPORT = "You have no new activity or submissions on Hold today!" diff --git a/tests/daily_update/test_process.py b/tests/daily_update/test_process.py index 724151e..74bacdc 100644 --- a/tests/daily_update/test_process.py +++ b/tests/daily_update/test_process.py @@ -125,7 +125,7 @@ def test_cross_does_not_reach_a_category_it_already_lives_in(sends): def test_mod_hold_lands_in_the_hold_section(sends): body = sends["digest-cat@example.com"]["body"] - hold_part = body.split("New:")[0] + hold_part = body.split("Scheduled for announcement:")[0] assert "submit/210" in hold_part #a rep on hold — the only way a replacement appears at all @@ -257,7 +257,7 @@ def test_the_whole_html_digest_for_one_moderator(sends): 'On Mod Hold
\n' 'Mod Hold Author
\n' 'Proposals: none

\n' - '

New:

\n' + '

Scheduled for announcement:

\n' '

07-27 18:00 EDT   cs.AI   Frank Franky   submit/213
\n' 'A Discussed Paper
\n' 'Talky Author
\n' diff --git a/tests/daily_update/test_report.py b/tests/daily_update/test_report.py index d3c58cb..317ccbf 100644 --- a/tests/daily_update/test_report.py +++ b/tests/daily_update/test_report.py @@ -52,7 +52,7 @@ def _mod(labels=("cs.AI",), categories=("cs.AI",)) -> DigestMod: # literals, so a reorder, a retitle or a dropped section fails here def test_sections_are_in_this_order_with_these_headings(): - assert [section.value for section in Section] == ["On Hold", "New", "Cross Lists"] + assert [section.value for section in Section] == ["On Hold", "Scheduled for announcement", "Cross Lists"] # ── header ────────────────────────────────────────────────────────────────── @@ -132,7 +132,7 @@ def test_a_section_with_content_still_shows_none_under_the_empty_ones(): "On Hold:\n" f" none\n" "\n" - "New:\n" + "Scheduled for announcement:\n" "a new one\n" "\n" "Cross Lists:\n" @@ -144,7 +144,7 @@ def test_a_section_with_content_still_shows_none_under_the_empty_ones(): assert body_html == ( "

On Hold:

\n" f"

none

\n" - "

New:

\n" + "

Scheduled for announcement:

\n" "

a new one

\n" "

Cross Lists:

\n" f"

none

\n" @@ -154,7 +154,7 @@ def test_a_section_with_content_still_shows_none_under_the_empty_ones(): def test_sections_appear_in_this_order(): entries = {section: [Rendered(f"{section.value} entry\n", "\n")] for section in Section} text, _ = render_body("cs.AI", entries) - positions = [text.index(t) for t in ("On Hold:", "New:", "Cross Lists:")] + positions = [text.index(t) for t in ("On Hold:", "Scheduled for announcement:", "Cross Lists:")] assert positions == sorted(positions)