Skip to content

docs: add investor workflow guides for growth, discovery, screening, underwriting - #336

Merged
paruff merged 4 commits into
mainfrom
docs/comprehensive-review
Aug 1, 2026
Merged

docs: add investor workflow guides for growth, discovery, screening, underwriting#336
paruff merged 4 commits into
mainfrom
docs/comprehensive-review

Conversation

@paruff

@paruff paruff commented Aug 1, 2026

Copy link
Copy Markdown
Owner

Summary

Comprehensive documentation review & revision for the Growth Areas → Discovery → Screening → Underwriting investor workflow (issue #335).

New guides

  • docs/explanation/investor-workflow.md — 4-stage navigation map
  • docs/how-to-guides/analyze-growth-areas.md — Growth Explorer step-by-step
  • docs/how-to-guides/discover-properties.md — source types + rent-data caveats
  • docs/how-to-guides/screen-properties.md — hard kills vs soft scoring
  • docs/how-to-guides/underwrite-deals.md — pipeline underwriting vs BRRRR
  • docs/reference/ui-patterns.md — design-system component reference

Updated

  • docs/explanation/GACS_GUIDE.mdfixed stale GACS v1 → v2 (7 signals, 0–100 scale, QCEW county employment with FRED fallback, weights 30/15/15/10/15/10/5) to match core/models/growth.py
  • docs/index.md, docs/README.md, section indexes, mkdocs.yml nav
  • Archived docs/implementation-summary-growth-areas.mddocs/assessments/

Validation

  • All new/updated doc links resolve
  • manage.py check clean; live smoke: all 6 documented pages return 200
  • Screening/underwriting/BRRRR/discovery claims cross-checked against core/services/*.py and templates
  • pre-commit all green

Notes

  • Docs-only change (no code)
  • Pre-existing: docs/explanation/index.md has 15 broken links to planned-but-unwritten pages (out of scope)

paruff added 4 commits August 1, 2026 14:01
Devcontainer failed to boot with 'No module named structlog' because
structlog was only a transitive dep of django-structlog and the
container's Python (3.14 from the 'latest' feature) predated it.

- Pin structlog==26.1.0 in requirements.txt (direct import in settings
  and middleware)
- Guard the structlog import in core/middleware.py to degrade to no-op
  logging instead of crashing the whole WSGI app (matches settings.py)
- Pin devcontainer python feature to 3.13 to match the base image
@paruff
paruff merged commit ff42608 into main Aug 1, 2026
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant