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

[HOTFIX]再生位置を指定するコードを移動させる #733

Merged

Conversation

y-chan
Copy link
Member

@y-chan y-chan commented Mar 1, 2022

内容

BlobとAudioElementだけで再生させられるようにリファクタを行いましたが、currentTimeをセットしてからsrcをセットするとcurrentTimeの方がリセットされてしまうようなので、そうならないようにcurrentTimeをセットするコードを移動させました。
しっかり動作確認せずにPRしてしまった私の問題です、ご迷惑をおかけしました...

関連 Issue

close #732

@y-chan y-chan requested a review from Hiroshiba March 1, 2022 13:06
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

LGTM!!

早速の修正助かります!! 1箇所だけコメントあったら意図が伝わりやすそうに感じたので追加してみました!!

src/store/audio.ts Show resolved Hide resolved
@Hiroshiba Hiroshiba merged commit 9b753ca into VOICEVOX:release-0.11 Mar 1, 2022
Hiroshiba added a commit that referenced this pull request Mar 1, 2022
* To 0.11.0 (#720)

* アプデ情報の情報整理

* GTM情報を書き換える

* デザイン調整

* アプデ情報追加

* update htu

* 辞書読み上げを最初のキャラに

* electron builderでは、.envファイルが読み込まれないっぽい?のでフォールバック (#722)

* Hotfix: バイナリでエンジン設定が読み出せず、エンジンが起動しない問題の原因を修正 (#721) (#723)

* use process.env.DEFAULT_ENGINE_INFOS after dotenv.config

* fix comment

* fix comment

* to 0.11.2 (#725)

* [HOTFIX]再生位置を指定するコードを移動させる (#733)

* move part of setting current time

* Update src/store/audio.ts

Co-authored-by: Hiroshiba <hihokaruta@gmail.com>

* [HOSTFIX] #730を修正 (#735)

* 辞書UIの状態遷移を整理 (#736)

* 辞書UIの状態遷移を整理

* closeDialog

* selectWord

* to-0.11.3

Co-authored-by: aoirint <aoirint@gmail.com>
Co-authored-by: Yuto Ashida <y-chan@y-chan.dev>
Co-authored-by: madosuki <yurikasamaaikatsu@gmail.com>
Hiroshiba added a commit that referenced this pull request Mar 19, 2022
* To 0.11.0 (#720)

* アプデ情報の情報整理

* GTM情報を書き換える

* デザイン調整

* アプデ情報追加

* update htu

* 辞書読み上げを最初のキャラに

* electron builderでは、.envファイルが読み込まれないっぽい?のでフォールバック (#722)

* Hotfix: バイナリでエンジン設定が読み出せず、エンジンが起動しない問題の原因を修正 (#721) (#723)

* use process.env.DEFAULT_ENGINE_INFOS after dotenv.config

* fix comment

* fix comment

* to 0.11.2 (#725)

* [HOTFIX]再生位置を指定するコードを移動させる (#733)

* move part of setting current time

* Update src/store/audio.ts

Co-authored-by: Hiroshiba <hihokaruta@gmail.com>

* [HOSTFIX] #730を修正 (#735)

* 辞書UIの状態遷移を整理 (#736)

* 辞書UIの状態遷移を整理

* closeDialog

* selectWord

* to-0.11.3

* to 0.11.4 (#760)

* updateinfoのミス修正 (#764)

Co-authored-by: aoirint <aoirint@gmail.com>
Co-authored-by: Yuto Ashida <y-chan@y-chan.dev>
Co-authored-by: madosuki <yurikasamaaikatsu@gmail.com>
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