Skip to content

過去のランキングページがランキングページと同じ表示になるのを修正 - #232

Merged
kojira merged 2 commits into
mainfrom
claude/ranking-feature-bug-a8nq2k
Aug 2, 2026
Merged

過去のランキングページがランキングページと同じ表示になるのを修正#232
kojira merged 2 commits into
mainfrom
claude/ranking-feature-bug-a8nq2k

Conversation

@kojira

@kojira kojira commented Aug 2, 2026

Copy link
Copy Markdown
Member

/ranking/history を開いても /ranking と全く同じ画面が出ていた。

原因

pages/ranking.vuepages/ranking/history.vue が共存していたため、Nuxt は ranking.vue親ルートhistory.vue子ルートとして構成する。子を描画するには親に <nuxt-child /> が必要だが、無かった。

結果、/ranking/history でもランキングページだけが描画され、履歴ページのコードは一度も実行されていなかった

これが「締めもアーカイブ保存も API も正常なのに履歴が出ない」の正体で、rankingArchiveGo にリクエストが1件も届いていなかったことの説明もつく。

修正

ranking.vueranking/index.vue に移して兄弟ルートにする。

確認したこと

ビルド後のルート定義が2つに分かれること:

path:"/ranking"
path:"/ranking/history"

反省

前回はこの機能を出すとき、yarn generatedist/ranking/history/index.html を作ったことだけを見て「ページ生成OK」としていた。SPA(ssr: false)では全ルートが同じシェルHTMLになるため、ファイルの存在は何の証明にもならない。ルート定義まで確認する必要があった。

🤖 Generated with Claude Code

https://claude.ai/code/session_01DwkB6Hj2JggoKcy7FoxNcU


Generated by Claude Code

kojira and others added 2 commits August 2, 2026 18:40
APIは正しく返っているのに画面に出ない、という状況では、配信されている
バンドルにその画面のコードが載っていない可能性がある。probe_url が返した
HTMLから読み込まれるJSを辿って、指定した文字列を含むか確かめられるようにする。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01DwkB6Hj2JggoKcy7FoxNcU
pages/ranking.vue と pages/ranking/history.vue が共存していたため、Nuxt が
ranking.vue を親ルート、history.vue を子ルートとして構成していた。親に
nuxt-child が無いので、/ranking/history を開いてもランキングページだけが
描画され、履歴ページのコードは一度も動いていなかった。

そのため rankingArchiveGo にはリクエストが1件も届かず、締めもAPIも正常なのに
「過去のランキングが出ない」状態になっていた。

ranking.vue を ranking/index.vue に移して兄弟ルートにする。生成されるルート
定義が /ranking と /ranking/history の2つになることを確認済み。

なお yarn generate は dist/ranking/history/index.html を作るが、SPAでは全ルートが
同じシェルHTMLになるため、ファイルの存在だけでは検証にならない。ルート定義まで
見る必要があった。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01DwkB6Hj2JggoKcy7FoxNcU
@cursor

cursor Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@kojira
kojira merged commit d78f3a2 into main Aug 2, 2026
3 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