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

ブラウザ版をデプロイ可能にする #1410

Merged
merged 4 commits into from
Jul 20, 2023

Conversation

Hiroshiba
Copy link
Member

内容

の解決です。実際にNetlifyを経由してデプロイしています。
https://voicevox-browser-dev.netlify.app/#/home

今はサンプルでこのブランチをデプロイしていますが、後でmainブランチに変えるつもりです。

関連 Issue

fix #1409

スクリーンショット・動画など

その他

@Hiroshiba Hiroshiba requested a review from a team as a code owner July 18, 2023 11:19
@Hiroshiba Hiroshiba requested review from y-chan and removed request for a team July 18, 2023 11:19
@netlify
Copy link

netlify bot commented Jul 18, 2023

Deploy Preview for voicevox-browser-dev ready!

Name Link
🔨 Latest commit 8fa450a
🔍 Latest deploy log https://app.netlify.com/sites/voicevox-browser-dev/deploys/64b67528550e280008da9a1a
😎 Deploy Preview https://deploy-preview-1410--voicevox-browser-dev.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.

@@ -4,6 +4,7 @@
<meta charset="utf-8" />
<meta http-equiv="X-UA-Compatible" content="IE=edge" />
<meta name="viewport" content="width=device-width,initial-scale=1.0" />
<meta name="robots" content="noindex,nofollow">
Copy link
Member Author

@Hiroshiba Hiroshiba Jul 18, 2023

Choose a reason for hiding this comment

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

あくまで今のところは開発版なので検索避けです。
将来本番デプロイして検索流入を許容する場合はここを省く必要があります。

Comment on lines 12 to 15
const router = createRouter({
history: isElectron
? createWebHashHistory(import.meta.env.BASE_URL)
: createWebHistory(import.meta.env.BASE_URL),
history: createWebHashHistory(import.meta.env.BASE_URL),
routes,
});
Copy link
Member Author

@Hiroshiba Hiroshiba Jul 18, 2023

Choose a reason for hiding this comment

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

ページにアクセスする時のURLがどういう形になるかの指定です。
元のブラウザ版だと https://host/home にアクセスを想定していて、変更後は https://host/index.html/#/home へのアクセスを想定します。

シングルページアプリケーションの場合はindex.htmlしかできないので /home のアクセスに失敗します。
それ用のリダイレクトやページを作成すればいいのですが、まあまだ本番でプロではないので一旦これでいいかなと思いました。

Copy link
Member

@y-chan y-chan left a comment

Choose a reason for hiding this comment

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

LGTMです!

@Hiroshiba
Copy link
Member Author

問題ないと思うのでマージします!!

@Hiroshiba Hiroshiba closed this Jul 20, 2023
@Hiroshiba Hiroshiba reopened this Jul 20, 2023
@Hiroshiba Hiroshiba merged commit 66cb02f into main Jul 20, 2023
@Hiroshiba Hiroshiba deleted the netlifyでデプロイする branch July 20, 2023 19:33
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.

開発中のブラウザ版をweb上で見れるようにする
2 participants