Skip to content

Updates - #262

Merged
gregv merged 4 commits into
mainfrom
develop
Aug 6, 2026
Merged

Updates#262
gregv merged 4 commits into
mainfrom
develop

Conversation

@gregv

@gregv gregv commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

No description provided.

gregv and others added 4 commits July 23, 2026 08:57
…roxy

Fix-forward for the project-page discoverability gap: approve_team created
the GitHub repo and wrote github_links on the team doc only, so problem
statements' github field stayed empty and /project/<id> showed no code.

- approve_team now also appends {name, link} to the github array of the
  team's linked problem statement(s) via _link_repo_to_problem_statements:
  team.problem_statements refs preferred (string ids and DocumentReferences
  both handled); falls back to the nonprofit's problem statement only when
  it has exactly one (ambiguous cases skip + log). Dedupes by normalized
  link, converts legacy string-shaped github values, clears the cached
  get_single_problem_statement_old read, and never blocks the approval flow.
- GET /api/github/issues responses are now cached 10 min per
  (org, repo, state) — the public project pages fetch this per repo, so the
  cache protects the shared GITHUB_TOKEN rate budget. Errors are not cached
  so transient GitHub failures retry.

Note: GITHUB_TOKEN currently returns 401 Bad credentials in prod — the
issues proxy (and admin issue summaries) won't work until it's rotated.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…res model

OpenAI removed the default model for images.generate, so sending a
certificate 400'd with "Missing required parameter: 'model'". Use
gpt-image-1 (matching common/utils/openai_api.py) and handle its
base64 response with a URL fallback.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…e-model

[Hearts] Fix certificate send: add required model param to OpenAI image call
…-to-problem-statements

Link team repos to problem statements on team approval; TTL-cache the GitHub issues proxy
@gregv
gregv merged commit 6a0528e into main Aug 6, 2026
12 of 13 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