Skip to content

D535 (#1081): record S1's decisions — currency 1A, third list 2A - #1306

Merged
guillaumelauzier merged 1 commit into
mainfrom
agent/claude-b/1081-d535-currency
Oct 6, 2026
Merged

guillaumelauzier merged 1 commit into
mainfrom
agent/claude-b/1081-d535-currency

Conversation

@guillaumelauzier

Copy link
Copy Markdown
Contributor

Objective

Record in D535 the two answers S1 gave on #1081 at 19:53Z, as S1 asked ("Record 1A in D535 now").

Part of #1081

Implementation

This changes documentation only. documentation/architecture/decisions/D535.md gets a new section, "Decided, 2026-10-06".

  • 1A, currency. A fund's size currency is derived from its list's name (EUR_SOURCE). There is no size_currency column and no migration. This replaces the earlier "currency column" item under "Not done here".
  • 2A, the third list. It is imported once its file is in the repository. The two ways to supply it are stated: a CSV export committed under attached_assets/, or allowing docs.google.com in the agents' environment.

The "Still open" line now names only the missing file.

Files changed

  • documentation/architecture/decisions/D535.md

Testing

  • node scripts/check-decision-ids.mjs: passes.
  • node scripts/check-folder-docs.mjs: passes.
  • npm run test:guards: exit 0.
  • No code changed, so there was nothing to mutation-check.

Risks

None: this is documentation only, and nothing ships.

Dependencies

#1081 stays blocked until the owner supplies the third list's file.

Agent

S09 · Claude Code (claude-b)

Review requested

S1.

🤖 Generated with Claude Code

https://claude.ai/code/session_01L8iwbvTs6nntzGe7MxmvUm


Generated by Claude Code

@github-actions

github-actions Bot commented Oct 6, 2026

Copy link
Copy Markdown

Preview: https://studioos-pr-1306.guillaumelauzier.workers.dev (built from cd6f0b4)

The pull request's SPA build on a Worker with no bindings: pages and deep links work, /api/* is a 404, and nothing here can reach production data. Redeployed on every push; deleted when the PR closes.

@guillaumelauzier
guillaumelauzier marked this pull request as ready for review October 6, 2026 21:58
@guillaumelauzier
guillaumelauzier merged commit c97fff2 into main Oct 6, 2026
21 checks passed
@guillaumelauzier
guillaumelauzier deleted the agent/claude-b/1081-d535-currency branch October 7, 2026 07:18
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.

2 participants