We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
electron-storeのマイグレーションは、まず過去のjsonオブジェクトを現在のjsonスキーマでバリデーションする仕様になっています。
electron-store
validate
この影響で、過去のjsonオブジェクトが現在のjsonスキーマに合わなかった場合、マイグレーション前にエラーが出ます。
なので、設定ファイルのマイグレーションはelectron-storeに頼らず、jsonオブジェクトを更新する形で自前実装したほうが楽になりそうです。
electron-storeの仕様に付き合わなくて良くなる
自前実装が必要
プロジェクトファイルのマイグレーションと同じ流れになりそうです。 こんな感じで、semverでsatisfyしてobjを書き換えるだけで良いかなと思います。
voicevox/src/store/project.ts
Lines 112 to 131 in 4a84ac7
問題としては、たしかelectron-storeは設定jsonオブジェクトを受け取れないはずです。 勝手にconfig.jsonを読み込むので、どのタイミングでどうやってマイグレーションした結果を反映すれば良いのかが若干見えないでいます。 (いっそelectron-storeから脱却したほうが良いかも・・・?)
The text was updated successfully, but these errors were encountered:
こちらのプルリクエストで解決しました!(thx @sevenc-nanashi !!!)
Sorry, something went wrong.
No branches or pull requests
内容
electron-store
のマイグレーションは、まず過去のjsonオブジェクトを現在のjsonスキーマでバリデーションする仕様になっています。validate
after migration. sindresorhus/conf#173この影響で、過去のjsonオブジェクトが現在のjsonスキーマに合わなかった場合、マイグレーション前にエラーが出ます。
なので、設定ファイルのマイグレーションはelectron-storeに頼らず、jsonオブジェクトを更新する形で自前実装したほうが楽になりそうです。
Pros 良くなる点
electron-store
の仕様に付き合わなくて良くなるCons 悪くなる点
自前実装が必要
実現方法
プロジェクトファイルのマイグレーションと同じ流れになりそうです。
こんな感じで、semverでsatisfyしてobjを書き換えるだけで良いかなと思います。
voicevox/src/store/project.ts
Lines 112 to 131 in 4a84ac7
問題としては、たしか
electron-store
は設定jsonオブジェクトを受け取れないはずです。勝手にconfig.jsonを読み込むので、どのタイミングでどうやってマイグレーションした結果を反映すれば良いのかが若干見えないでいます。
(いっそ
electron-store
から脱却したほうが良いかも・・・?)The text was updated successfully, but these errors were encountered: