Drop "End of Month Campaign Report" suffix from CTA Campaign headers
CTA phases now get the same plain "TXproducts — {timeframe}" header
every non-branding-campaign period already uses, instead of the
branding campaign's own "...End of Month Campaign Report" phrasing —
the Campaign dropdown above it already names the campaign, no need to
repeat it in the header too.
This commit is contained in:
1 parent
03e6d7287e
commit
91b9f3f232
2 files changed
+11
-9
No files matched your search
@@ -4530,12 +4530,12 @@ def _render_reports_tab(month_value):
|
||||
notes = monthly_report_notes.NOTES.get(month_value)
|
||||
fts_f, posts_f = stats["fts_f"], stats["posts_f"]
|
||||
|
||||
# Only July/August/September were part of the 3-month branding campaign, and
|
||||
# CTA Campaign phases (CUSTOM_PERIODS) once that campaign is live — any other
|
||||
# month like October gets a plain title, no active campaign stage
|
||||
# (goal_line/core_message/roadmap) behind it.
|
||||
_is_campaign_month = month_value in report.CUSTOM_PERIODS
|
||||
if not _is_campaign_month:
|
||||
# Only July/August/September were part of the 3-month branding campaign — any
|
||||
# other month (October, or a CTA Campaign phase) gets a plain title, just the
|
||||
# timeframe, no "...Campaign Report" suffix repeating what the dropdown above
|
||||
# it already names.
|
||||
_is_campaign_month = False
|
||||
if month_value not in report.CUSTOM_PERIODS:
|
||||
try:
|
||||
_year, _month = (int(p) for p in month_value.split("-"))
|
||||
_month_start = pd.Timestamp(year=_year, month=_month, day=1)
|
||||
|
||||
@@ -271,9 +271,11 @@ def build_report_docx(ctx, timeframe_value="all"):
|
||||
# (CAMPAIGN_START-CAMPAIGN_END) — a later month like October gets a plain
|
||||
# title, not "...Campaign Report", since there's no active campaign stage
|
||||
# (no goal_line/core_message/roadmap) behind it. CTA Campaign phases
|
||||
# (CUSTOM_PERIODS) get the same campaign-report treatment once that
|
||||
# campaign is approved and running.
|
||||
is_campaign_month = timeframe_value in CUSTOM_PERIODS
|
||||
# (CUSTOM_PERIODS) deliberately do NOT get this suffix — their header is
|
||||
# just the timeframe, same plain treatment as any other non-branding-
|
||||
# campaign period (the dropdown already names the campaign, no need to
|
||||
# repeat "Campaign Report" in the header too).
|
||||
is_campaign_month = False
|
||||
if timeframe_value and timeframe_value != "all" and not timeframe_value.startswith("year-") \
|
||||
and timeframe_value != "campaign-end" and timeframe_value not in CUSTOM_PERIODS:
|
||||
try:
|
||||
|
||||
Reference in new issue
Block a user