Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

apply LocaleSwitch #144

Merged
merged 2 commits into from
May 29, 2024
Merged

apply LocaleSwitch #144

merged 2 commits into from
May 29, 2024

Conversation

jiyuujin
Copy link
Collaborator

No description provided.

Copy link

netlify bot commented May 24, 2024

Deploy Preview for vuefes-2024 ready!

Name Link
🔨 Latest commit 0e0bf5b
🔍 Latest deploy log https://app.netlify.com/sites/vuefes-2024/deploys/6653d9eb321a7900086e8338
😎 Deploy Preview https://deploy-preview-144--vuefes-2024.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@totocalcio
Copy link
Contributor

totocalcio commented May 25, 2024

Deploy Previewで確認したところ動作が機能要件を満たしていないようです。

概要

サブページで言語を切り替えてトップページへ遷移、その後言語を切り替えても、切り替えできない

再現手順

  1. プライバシーポリシー、もしくは行動規範のページに遷移する
  2. 言語切り替えスイッチでenへ切り替え
  3. ロゴをクリックしてTOPページへ遷移
  4. 言語切り替えスイッチでjaに切り替え(言語は切り替わらない)
  5. 再度言語切り替えスイッチでenに切り替え
    →ページが見つからない

原因

  • トレイリングスラッシュがないとパスの切り替えがうまくいかないため。
  • enからjaのパスの切り替えは/en/を置換しているため

詳細

  • TOPページはトレイリングスラッシュあり
  • サブページはトレイリングスラッシュなし
  • サブページで言語を切り替えると/privacyから/en/privacyにパスが切り替わる
  • 言語がen、かつサブページからTOPページへ遷移するとURLは/2024/enと、トレイリングスラッシュなしになる
  • 言語を切り替えようとしても/en/が見つからず内部の状態だけ日本語になり、パスは置換されない。
  • 内部状態は日本語なので再度言語切り替えを行うと/2024/en/enとなる

対応方針案

  • サーバー設定で必ずトレイリングスラッシュありにする
  • 置換対象を/en/ではなく/enとする
    • /enhance などのパスの場合一致してしまう可能性があるが、Vue Fes 2024サイトで言えばないと割り切ることは可能。

その他調査内容

本番環境でもトレイリングスラッシュあり・なしは同じ挙動となっていることを確認済み。

TOPページ

https://vuefes.jp/2024をアドレスバーに入力
→トレイリングスラッシュあり:https://vuefes.jp/2024/

サブページ

https://vuefes.jp/2024/privacy/をアドレスバーに入力
→トレイリングスラッシュなし:https://vuefes.jp/2024/privacy

@@ -94,6 +94,14 @@ export default defineNuxtConfig({
const supabaseKey = process.env.SUPABASE_KEY
if (!supabaseUrl || !supabaseKey) return
},
'prerender:routes': (context) => {
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✍️

trailingSlash 設定を書いています mm

@jiyuujin
Copy link
Collaborator Author

サーバー設定で必ずトレイリングスラッシュありにする
置換対象を/en/ではなく/enとする

この辺りは対応させていただきました cc: @totocalcio

@jiyuujin
Copy link
Collaborator Author

一旦マージします(何か問題あればリバートします

@jiyuujin jiyuujin merged commit 218c1f4 into main May 29, 2024
5 of 6 checks passed
@jiyuujin jiyuujin deleted the enhance/apply-locale-switch branch May 29, 2024 01:02
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