オフラインでも時間割を確認できるように #601
オフラインでも時間割を確認できるように #601
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
| if (process.env.NODE_ENV === "development") { | ||
| base_url = process.env.BASE_URL || "http://localhost:4000"; | ||
| } else { | ||
| base_url = process.env.BASE_URL; |
There was a problem hiding this comment.
TODO: 本番ビルド時に環境変数を注入するように設定をする
Production で dotenv （に限らずコミットされるファイルに環境値がかかれたもの）を使用するのは微妙だと思うため。
なぜならアプリが環境を意識しないための環境変数なのに、環境（ファイル）を意識しては元も子もない。
HosokawaR
force-pushed
the
feature/offline-mode
branch
from
fb14363 to
ad4cc03
Compare
| { | ||
| "include": ["./**/*.ts"], | ||
| "compilerOptions": { | ||
| "sourceMap": true, | ||
| "module": "esnext", | ||
| "moduleResolution": "node", | ||
| "preserveValueImports": false, | ||
| "esModuleInterop": true, | ||
| "importsNotUsedAsValues": "remove", | ||
| "baseUrl": ".", | ||
| "paths": { | ||
| "~/*": ["./*"] | ||
| }, | ||
| "lib": ["ES2021", "WebWorker"], | ||
| "types": ["node"], | ||
| "outDir": "../dist" | ||
| } | ||
| } |
There was a problem hiding this comment.
/src 以下のページスクリプトと /serviceworker は別世界なのでそれぞれ独自に tsconfig を持ちたい。
例えば /serviceworkers/tsconfig.ts が "extends": "@vue/tsconfig/tsconfig.web.json" という設定値を持ったり、
/tsconfig.ts が "lib": ["ES2021", "WebWorker"], という設定値をもつのはおかしい
| /** unregister service worker (v2) */ | ||
| onMounted(() => { | ||
| navigator.serviceWorker.getRegistrations().then(function (registrations) { | ||
| for (let registration of registrations) { | ||
| registration.unregister(); | ||
| } | ||
| }); | ||
| }); |
There was a problem hiding this comment.
v2 公開終了から 1 年半が経過したので流石にもういらないと思われる。
| request.url.startsWith(base_url) || | ||
| request.url.startsWith("https://fonts.googleapis.com/") || | ||
| request.url.startsWith("https://fonts.gstatic.com/") |
There was a problem hiding this comment.
Google Fonts がないとオフライン時にアイコンが表示されないのでキャッシュをしている。
| @@ -5,7 +5,9 @@ const tsconfigFilename = "tsconfig.staged.json"; | |||
|
|
|||
| const typecheckOnlyStaged = (stagedFilenames) => { | |||
| const tsconfig = JSON.parse(fs.readFileSync("tsconfig.json")); | |||
| tsconfig.include = stagedFilenames; | |||
| tsconfig.include = stagedFilenames.filter((filename) => | |||
| filename.includes("/src/") | |||
There was a problem hiding this comment.
/tsconfig.ts に "include": ["src/**/*.ts", "src/**/*.d.ts", "src/**/*.tsx", "src/**/*.vue"],が指定されていることからわかるように、typecheckOnlyStagedは /src 以下の型検査のみに関心を持っている。したがってここでは/src以下を検査するようにして /serviceworkers 以下は除外している。
なお yarn typecheck で /serviceworkers 以下の型検査はちゃんと走る。
|
再開の機運がないのでこの PR は一旦閉じる |
Resolves #594
基本的な機能は完成したが、以下が決まっていないので Draft
またビルド構成なども乱雑なので議論の余地あり
#600 にあるように、CORS 無効化の拡張機能では動作確認ができません。
変更点