fix: すべてのURLを書き直していた問題を修正#467
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughdev環境でのタイルサーバーURL書き換え処理に対して、ホスト名がtileserver.geolonia.comの場合のみtileserver-dev.geolonia.comへの書き換えを実行するよう条件を追加しました。 Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
@bougan1160
現状の最新 master ブランチでE2Eテスト実行時(npm run e2e)にhttps://tileserver-dev.geolonia.com/tiles.json にアクセスして404エラーが発生する問題が、こちらの変更により解消されることを確認しましたので、Approveします。
なお、私のローカル環境では以下のExternal Style Support関連のエラーがまだ発生するようでしたので、念のため報告します。
6 failed
[chromium] › e2e/external-style.spec.ts:7:7 › External Style Support › data-style で外部スタイルを指定しても地図が表示されること
[chromium] › e2e/external-style.spec.ts:15:7 › External Style Support › data-style で指定した外部スタイルURLにアクセスしていること
[firefox] › e2e/external-style.spec.ts:7:7 › External Style Support › data-style で外部スタイルを指定しても地図が表示されること
[firefox] › e2e/external-style.spec.ts:15:7 › External Style Support › data-style で指定した外部スタイルURLにアクセスしていること
[webkit] › e2e/external-style.spec.ts:7:7 › External Style Support › data-style で外部スタイルを指定しても地図が表示されること
[webkit] › e2e/external-style.spec.ts:15:7 › External Style Support › data-style で指定した外部スタイルURLにアクセスしていること
15 passed (48.8s)
Fixes #465
Summary
Checklist (optional)
Summary by CodeRabbit
バグ修正