Skip to content

fix(web): 自治体の階層表示とチャット起動を修正 - #316

Merged
wwwyo merged 1 commit into
mainfrom
wwwyo/fix-jurisdiction-axis-brand
Oct 22, 2026
Merged

wwwyo merged 1 commit into
mainfrom
wwwyo/fix-jurisdiction-axis-brand

Conversation

@wwwyo

@wwwyo wwwyo commented Oct 22, 2026

Copy link
Copy Markdown
Owner

Blast Radius

中 — 自治体詳細の表示、階層取得、チャット起動に影響します。実API・型検査・テスト・ビルドで検証済みです。

What

北区に収録済みの決算が表示されない問題と、自治体ページの提案タグからチャットを開始できない問題を修正します。流れ図は全款・全項を初期表示し、目・節も全体で深さを切り替えられるようにします。

Background

#310 で追加した自治体詳細を、自治体ごとの科目軸や収録階層の違いに対応させます。このPRでは表示とチャット起動の不具合を修正します。

QA

  • 型検査、webテスト35件、availabilityテスト4件、フォント検査、webビルド成功。
  • 北区の実APIで11款・41項・117目・933節を取得。
  • 独立レビューで北区・中野区・立川市・利島村・神戸市の階層を検証。P1以上なし。
  • ブラウザでの描画と認証完了は未確認。八王子の未記載項金額が0円になる既存処理は後続修正します。

Summary

  • 科目軸の物理名ではなく共通の科目判定で検索し、全ページを取得。
  • 全款を表示する深さ切替とスクロール可能な流れ図。
  • 自治体詳細をゴシック体に統一し、正式なロゴに復元。
  • 自治体ページでもTurnstileを読み込み、提案タグからのチャット起動を修正。

@wwwyo wwwyo self-assigned this Oct 22, 2026
@pullfrog

pullfrog Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

This run was cancelled before it finished.

Pullfrog  | View workflow run | via Pullfrog

@coderabbitai

coderabbitai Bot commented Oct 22, 2026

Copy link
Copy Markdown

Review in Change Stack →

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
📥 Commits

Reviewing files that changed from the base of the PR and between ff236c0 and 02eba4a.

📒 Files selected for processing: 12
  • apps/web/chat/components/ui/toggle-group.tsx
  • apps/web/chat/components/ui/toggle.tsx
  • apps/web/chat/turnstile.ts
  • apps/web/jurisdiction/html.ts
  • apps/web/jurisdiction/jurisdiction.css
  • apps/web/jurisdiction/jurisdiction.tsx
  • apps/web/jurisdiction/sankey.tsx
  • apps/web/jurisdiction/types.ts
  • apps/web/landing.ts
  • apps/web/server/jurisdiction-routes.ts
  • apps/web/server/jurisdiction.test.ts
  • apps/web/server/jurisdiction.ts
+------------------------------------------------------------------------+
| OpenAI said I could be anything I wanted, so I became a code reviewer. |
+------------------------------------------------------------------------+
  \
   \     /)/)
        ( '')
        /  |
      o(___)
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@wwwyo
wwwyo merged commit d239d9a into main Oct 22, 2026
5 of 7 checks passed
@wwwyo
wwwyo deleted the wwwyo/fix-jurisdiction-axis-brand branch October 22, 2026 10:57
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