d7015b9210
Konrad's decision (15 May 2026): Phase A.2 (manual JournalEntry
UI) + Phase B (Letterly inbound webhook) are too complex to
interleave with normal app work β they'll be built and tested
offline on a separate track, not in ai-dev.
Verified there is NOTHING to remove or bypass: zero JournalEntry
model/views/urls/templates, zero Letterly/webhook/@csrf_exempt
code anywhere on ai-dev, latest migration is 0015. The working
app was already 100% clean of journal/voice code β these features
never left the design-doc stage.
Doc changes:
- parked-work.md: "Blocked on Konrad's input" section replaced
with "π§ Backburner β separate offline track", with an explicit
"do NOT start in ai-dev" warning and the nothing-to-remove
verification recorded.
- CLAUDE.md breadcrumb: reframed from "parked pending Q5/Q7
answers" (implies ready-to-go once answered) to "deliberately
deferred to offline track β do not pick up as normal feature
work".
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
191 lines
9.5 KiB
Markdown
191 lines
9.5 KiB
Markdown
# Parked / deferred work
|
||
|
||
> Updated 15 May 2026. A small index of features that are designed,
|
||
> half-built, blocked on input, or pending an operator step. When a
|
||
> fresh session opens, glance here first to see what's already on
|
||
> the workbench.
|
||
|
||
---
|
||
|
||
## Production status β β
fully caught up (15 May 2026)
|
||
|
||
Production (`https://foxlog.flatlogic.app/`) is deployed at
|
||
`origin/ai-dev` HEAD **`1d224bc`**. Everything is live: SiteReport
|
||
+ Absences feature, all UX polish, the team filters, the
|
||
dashboard/report number audit (18 findings), and the
|
||
template-comment + subline-clarity fixes.
|
||
|
||
**Deploy-ordering lesson learned (now in CLAUDE.md):** production
|
||
runs `DEBUG=False` β Django's cached template loader holds compiled
|
||
templates in memory until the process restarts. The 14 commits of
|
||
template fixes were briefly "invisible" because the service was
|
||
restarted *before* the code reached the target commit. The fix was
|
||
simply to restart AGAIN once `git log` confirmed the right commit.
|
||
Correct order is always: `git reset --hard github/ai-dev` β migrate
|
||
(if needed) β collectstatic (if `static/` changed) β restart
|
||
**last**. See the "DEPLOY ORDERING" bullet in CLAUDE.md's
|
||
Flatlogic Deployment section.
|
||
|
||
---
|
||
|
||
## π§ Backburner β journal/voice features (separate offline track)
|
||
|
||
> **Decision (15 May 2026):** Konrad parked the entire journal /
|
||
> voice-transcript line of work. It's more complex than the rest of
|
||
> the app and will be built + tested **offline, on a separate
|
||
> track**, NOT mixed into `ai-dev`. **Do NOT start either of these
|
||
> in `ai-dev` or in any normal feature session.** They are not
|
||
> "blocked waiting on an answer" β they are deliberately deferred
|
||
> until Konrad chooses to spin up a dedicated offline effort.
|
||
|
||
**Important β nothing is in the working app.** Verified 15 May 2026:
|
||
zero `JournalEntry` model / views / URLs / templates anywhere in
|
||
`core/`, zero Letterly / webhook / `@csrf_exempt` code on `ai-dev`,
|
||
latest migration is `0015_absence_project`. There is no dead code,
|
||
no feature flag, no migration to reverse. "Removing" these was a
|
||
no-op β they were never built into the running app. The working
|
||
app (`ai-dev` + production) is completely clean of journal/voice
|
||
code.
|
||
|
||
### Phase A.2 β Manual JournalEntry web UI (deferred)
|
||
|
||
Designed in the Site Work Logging brainstorm (`~/.claude/plans/
|
||
prancy-painting-brook.md`, local). A `JournalEntry` model + manual
|
||
entry form/list/edit, admin-only. Open question if/when resumed:
|
||
**Q7 "Who is Vi?"** β drives whether `JournalEntry` needs a
|
||
`recipient`/`audience` field. Default if never answered: no
|
||
recipient field, all-admin-readable. ~1 hour scope. When resumed,
|
||
build on the offline track first, not directly in `ai-dev`.
|
||
|
||
### Phase B β Letterly inbound webhook (deferred)
|
||
|
||
Designed in the same brainstorm. A `@csrf_exempt` view at
|
||
`/webhooks/letterly/` that creates `JournalEntry` rows from voice
|
||
transcripts, with HMAC body verification + shared-secret URL token.
|
||
Open question if/when resumed: **Q5 β a sample Letterly webhook
|
||
payload** (does it include the recording user / project metadata /
|
||
audio URL / custom fields?). ~3-4 hours scope + one-off Letterly
|
||
account setup. This was always meant to live on the `integrations`
|
||
branch, never `ai-dev` β and now it's offline-track until Konrad
|
||
revisits.
|
||
|
||
---
|
||
|
||
## Small polish follow-ups from the Absences feature
|
||
|
||
> All 7 cleared on 15 May 2026 (commit `d1d3e15`). Kept the section
|
||
> header in place because more polish items will accumulate from
|
||
> future code reviews β when they do, append here. The shipped items
|
||
> are summarized under "Recently shipped" below.
|
||
|
||
---
|
||
|
||
## Defaulted (not blocking β flag if you disagree)
|
||
|
||
From Q9, Q4 of the Site Work Logging brainstorm:
|
||
|
||
- **Q9 β photos on site reports.** Deferred to v2 of SiteReport.
|
||
Construction supervisors often want to attach a phone photo of
|
||
"plinths cast today" but it adds ~50 LOC + storage handling +
|
||
thumbnail rendering. v1 ships without.
|
||
- **Q4 β per-project metric templates.** Same metric set for all
|
||
projects in v1. If/when a non-solar-farm project lands and the
|
||
metrics diverge wildly, we add a `MetricTemplate` model. YAGNI
|
||
for now β `core/site_report_schema.py` is one Python file edit.
|
||
|
||
---
|
||
|
||
## Recently shipped (for context, so a fresh session knows what just landed)
|
||
|
||
- **Dashboard + report numbers audit** (commits `18c75b2` β `c02edce`,
|
||
15 May 2026): Konrad spotted that the home dashboard's "Paid This
|
||
Month" actually showed the last 60 days. A thorough audit found
|
||
17 more issues across the home dashboard, payroll dashboard, and
|
||
report. All 18 fixed in 7 commits: calendar-month math, outstanding
|
||
totals aligned between dashboards, `Project.name` dedupe (was
|
||
silently merging same-named projects), `timezone.localdate()`
|
||
replacing `timezone.now().date()` (SAST drift fix), off-by-one
|
||
on "last 7 / 60 days" windows, dead `year_projects`/`year_teams`
|
||
context removed, `_company_cost_velocity` rewritten as single SQL
|
||
aggregate, signed/coloured adjustment amounts on report tables
|
||
(Bonus vs Deduction now visually distinct), "Outstanding Now"
|
||
hero card subline now shows scope when filters active, dual
|
||
daily_rate code paths documented in CLAUDE.md. 160 β 173 tests
|
||
(+13 regression).
|
||
- **Absence-feature polish pass** (commit `d1d3e15`, 15 May 2026):
|
||
Seven small cleanups in one focused commit. (1) Deleted unused
|
||
`AbsenceQuickForm` (Round C made it obsolete). (2) Extracted
|
||
`_build_team_workers_map(user)` helper using `Prefetch(..., to_attr=...)`,
|
||
fixing N+1 on both `attendance_log` and `absence_log` (one query
|
||
instead of one-per-team). (3) `absence_list` now calls the existing
|
||
`_user_can_log_absences` helper. (4) Removed undefined
|
||
`var(--badge-neutral-bg)` CSS wrapper β fallback hex was always
|
||
winning anyway. (5) `conflicting_worklogs()` rewritten as a single
|
||
query with Python-side pair-set filtering β 25-query β 2-query for
|
||
a 5Γ5 form. (6) Extracted `_apply_absence_filters(qs, request)`
|
||
shared by `absence_list` + `absence_export_csv` so adding a filter
|
||
param now touches one place, not two. (7) `site_report_detail.html`
|
||
paid-check icon uses `text-success` instead of background-color
|
||
variable as foreground (WCAG contrast fix). 157/157 tests still
|
||
green.
|
||
- **Day name in Work Log Payroll modal header** (commit `70fa085`,
|
||
15 May 2026): The modal opened from /history/ now shows
|
||
"Friday, 15 May 2026" instead of "2026-05-15". Pure server-side
|
||
`strftime` on the already-loaded `log.date` β zero overhead.
|
||
- **Team filters on `/workers/` and `/history/`** (commits `4b57cff`,
|
||
`398a5b2`, 15 May 2026): Both pages now have a Team dropdown in
|
||
their filter row. `/workers/?team=<id>` filters by Team.workers
|
||
M2M membership (who's CURRENTLY on the team); `/history/?team=<id>`
|
||
filters by WorkLog.team FK (logs TAGGED with the team when
|
||
created β different semantics, intentional). Both accept
|
||
`team=none` for the "no team assigned" / "ad-hoc work log" case.
|
||
CSV export at `/history/export/` honours the same param.
|
||
+8 regression tests.
|
||
- **Absences UX polish** (commits `4368e53`, `02c6d4d`, 15 May 2026):
|
||
Two production-found bugs fixed. (1) Absence-form team filter was
|
||
hiding ALL workers when a team was selected β switched the JS to
|
||
read `<input name="workers">[value]` directly instead of going
|
||
through `data-worker-id` (proven attendance-form pattern). (2)
|
||
Reasons multi-checkbox dropdown on `/absences/` was rendering
|
||
behind the table β lifted the filter card with
|
||
`position: relative; z-index: 10` so the whole card sits above
|
||
its sibling table card in the stacking order. See the Coding
|
||
Style section in CLAUDE.md for both gotchas.
|
||
- **Worker Absences feature** (commits `bf6f0a5` β `27fe05e`,
|
||
14 May 2026): Complete absence-tracking system. 8 reason choices,
|
||
optional project FK (auto-attributes paid-absence Bonus
|
||
adjustments to the project), date-range logging with weekend
|
||
toggles, supervisor scoping (admin sees all; supervisors see
|
||
their teams). Standalone `/absences/log/` form + "Submit + Log
|
||
Absences" shortcut on attendance form. List/edit/delete + CSV
|
||
export. Worker-detail "Absences" tab with YTD totals. Dashboard
|
||
alert card "X absent in last 7 days". Bidirectional cascade with
|
||
PayrollAdjustment via `_sync_absence_payroll_adjustment` helper
|
||
(single-chokepoint design, transaction.atomic-wrapped). 12
|
||
commits pushed to `origin/ai-dev`. Test count: 85 β 149 (+64
|
||
tests). Migrations `0013_add_site_report`, `0014_add_absence`,
|
||
`0015_absence_project` are LIVE on production as of late 14 May
|
||
2026.
|
||
- **Phase A.1 β SiteReport** (commit `864ae72`, 14 May 2026):
|
||
Model, migration `0013_add_site_report.py`, form, two-step flow
|
||
from attendance log, 16 new tests. `CLAUDE.md` updated with model
|
||
summary + `site_report_schema.py` pattern section.
|
||
- **Pastel soft-fill payroll action buttons** (commit `6c6ade9`,
|
||
24 Apr 2026): unified Worker Lookup / Batch Pay / Add Adjustment
|
||
/ Price Overtime treatment.
|
||
- **Path A display-only rename** (UX Polish Pass, Apr 2026):
|
||
"New Loan" / "Advance Payment" / "Advance Repayment" display as
|
||
"Loan" / "Advance" / "Advance Repaid" β but **DB values are
|
||
unchanged**. See "UI-vs-DB naming drift" section in `CLAUDE.md`
|
||
before writing any filter/comparison on `PayrollAdjustment.type`.
|
||
|
||
---
|
||
|
||
## Original Site Work Logging brainstorm
|
||
|
||
The full Q1-Q13 question list and design rationale lives in
|
||
`~/.claude/plans/prancy-painting-brook.md` (local β deliberately
|
||
not committed to the repo, since the parts that matter have been
|
||
absorbed into `CLAUDE.md` and this file). If Konrad answers Q5 or
|
||
Q7 later, refer back to that file for the original framing.
|