chore: 本番URLを furdrop.app に変更 + LPヒーロー画像のプレースホルダー差し替え - #81
Merged
Conversation
本番ドメインを独自ドメイン furdrop.app へ移行する。 - .env.prod: VITE_PUBLIC_HOST を furdrop.app に更新 (dotenvx)。 index.html の %VITE_PUBLIC_HOST% (canonical / OGP / Schema.org) は generate:prod + Vite ビルドで自動置換される (dist で確認済み) - workers/r2-cors.json: R2 の許可オリジンを furdrop.app に変更 - public/sitemap.xml, public/robots.txt: 静的ファイルのため直接更新 - scripts/generate-og.py + public/og.png: OG画像のURLバッジを再生成 - GuidePage / LandingPage / ShotsPage: VITE_PUBLIC_HOST 未設定時の フォールバック値と EXIF デモ表示を furdrop.app に更新 - marketing/release-trailer: Scene3 の表示URLが furdrop.app なのに QRコードは furdrop.pages.dev を指していた不整合を解消 (QRも furdrop.app へ) Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01J9Q5inTn4HLVjmmdHXxDt8
📝 WalkthroughWalkthrough公開ホスト参照を Changes公開ホスト更新
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@marketing/release-trailer/index.html`:
- Line 46: The QR code target is inconsistent with the displayed destination in
release-trailer, so update the `data=` value in the QR code URL to match the
visible `furdrop.app/send/` destination. Make the same fix in both places
referenced by the QR generation markup so the `href` content and the on-screen
URL stay aligned.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: dcd1be03-7397-4f20-8e7d-0cfbf6cf8f3e
⛔ Files ignored due to path filters (1)
frontend/public/og.pngis excluded by!**/*.png
📒 Files selected for processing (9)
.env.prodfrontend/public/robots.txtfrontend/public/sitemap.xmlfrontend/scripts/generate-og.pyfrontend/src/pages/GuidePage.tsxfrontend/src/pages/LandingPage.tsxfrontend/src/pages/__shots__/ShotsPage.tsxmarketing/release-trailer/index.htmlworkers/r2-cors.json
トップのヒーローにある PolaroidPhoto 4枚のダミー (CSS グラデーション縞模様) を、release-trailer と同じ Lorem Picsum (picsum.photos) の画像に変更する。 - gradientAngle プロップを廃止し seed を追加、object-cover で実画像を表示 - シードは release-trailer と統一 (mountain1 / ocean1 / forest1 / sunset1) - 下部の手書き風キャプション (ファイル名) は維持 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01J9Q5inTn4HLVjmmdHXxDt8
本番 (.env.prod) に合わせ、開発用 .env のホストも furdrop.app に揃える。 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01J9Q5inTn4HLVjmmdHXxDt8
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
本番ドメインを Cloudflare Pages のデフォルト
furdrop.pages.devから独自ドメインfurdrop.appへ移行し、あわせて LP ヒーローのプレースホルダー画像を release-trailer と統一します。1. 本番URL変更
furdrop.pages.dev→furdrop.appビルド時に自動置換される箇所
.env.prod/.env:VITE_PUBLIC_HOSTをfurdrop.appに更新(本番・開発とも、dotenvx 再暗号化)。index.htmlの%VITE_PUBLIC_HOST%(canonical / OGP / Schema.org の計6箇所)はgenerate:prod→ Vite ビルドで置換され、dist/index.htmlでhttps://furdrop.appに解決されることを確認済み直接更新が必要な箇所(自動置換されない)
workers/r2-cors.json: R2 バケットの許可オリジンをhttps://furdrop.appに変更(CI のcors:prodで適用)frontend/public/sitemap.xml/robots.txt: 静的ファイルのため直接更新frontend/scripts/generate-og.py+frontend/public/og.png: OG 画像下部の URL バッジをfurdrop.appに変更し、画像を再生成GuidePage.tsx/LandingPage.tsx/ShotsPage.tsx:VITE_PUBLIC_HOST未設定時のフォールバック値、および EXIF デモ表示をfurdrop.appに更新リリースPVの不整合修正
marketing/release-trailer/index.html: Scene 3 の表示 URL は既にfurdrop.app/send/だったが、QRコードの遷移先がfurdrop.pages.devのままで不一致だった。QR(preload / 実描画 / alt)をfurdrop.appに統一2. LP ヒーロー画像のプレースホルダー差し替え
frontend/src/pages/LandingPage.tsx: トップのヒーローにあるPolaroidPhoto(4枚)のダミー(CSS グラデーション縞模様)を、release-trailer と同じpicsum.photos(Lorem Picsum) の画像に差し替えgradientAngleプロップを廃止しseedを追加、object-coverで実画像を表示mountain1/ocean1/forest1/sunset1)デプロイ後の手動確認が必要な項目(リポジトリ外)
furdrop.appを割り当てfurdrop.appを追加(Twitter/Google OAuth のリダイレクトに必要)furdrop.appで適用されたか(CIcors:prod後)Test plan
furdrop.appで送信フロー(URL発行→アップロード→受信ギャラリー表示)が通るfurdrop.appから R2 へのアップロード(PUT)が CORS で弾かれないfurdrop.appに遷移する🤖 Generated with Claude Code
https://claude.ai/code/session_01J9Q5inTn4HLVjmmdHXxDt8