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

コンポーネント群の修正 #1681

Merged
merged 2 commits into from
Dec 30, 2023

Conversation

Tksn07
Copy link
Contributor

@Tksn07 Tksn07 commented Dec 28, 2023

内容

  • components群のwarningが出ていた箇所を緩い等価比較に修正

関連 Issue

ref #1513

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

  • src/components/AccentPhrase.vue
    変更前後で挙動に変動がないことを確認
Screenshot 2023-12-27 at 22 22 26
  • src/components/AudioInfo.vue
    number型のためundefinedになることはないが、既存実装を優先して == に修正
image
  • src/components/CharacterOrderDialog.vue
    変更前後で挙動に変動がないことを確認
Screenshot 2023-12-30 at 1 06 50
  • src/components/CharacterPortrait.vue
    変更前後で挙動に変動がないことを確認
Screenshot 2023-12-30 at 1 12 33
  • src/components/DefaultStyleListDialog.vue
    変更前後で挙動に変動がないことを確認
Screenshot 2023-12-30 at 1 46 05
  • src/components/DefaultStyleSelectDialog.vue
    変更前後で挙動に変動がないことを確認
Screenshot 2023-12-30 at 1 52 35
  • src/components/Dialog.ts
    == で挙動上問題なし
image
  • src/components/EngineManageDialog.vue
    変更前後に挙動に変動がないことを確認
Screenshot 2023-12-28 at 14 13 28
  • src/components/FileNamePatternDialog.vue
    変更前後に挙動に変動がないことを確認
Screenshot 2023-12-28 at 13 25 37
  • src/components/HeaderBarCustomDialog.vue
    ref参照によりundefinedの可能性があるが、基本10個のボタンが選択されるため、挙動自体に問題なし
image
  • src/components/HotkeySettingDialog.vue
    変更前後で挙動に変動がないことを確認
Screenshot 2023-12-28 at 10 52 32
  • src/components/MenuBar.vue
    変更前後で挙動に変動がなく正常にことを確認
Screenshot 2023-12-27 at 23 20 22 Screenshot 2023-12-27 at 23 21 12

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.

丁寧な検証ありがとうございます!!!
ドラフトになっていますがもうレビューしても大丈夫でしょうか?

src/components/HeaderBarCustomDialog.vue Outdated Show resolved Hide resolved
@Tksn07
Copy link
Contributor Author

Tksn07 commented Dec 29, 2023

@Hiroshiba コメントありがとうございます!

  • src/components/AudioInfo.vue
  • src/components/CharacterOrderDialog.vue
  • src/components/CharacterPortrait.vue
  • src/components/DefaultStyleListDialog.vue
  • src/components/DefaultStyleSelectDialog.vue
    こちらのコンポーネント群がまだ修正&確認が済んでいないので完了次第Openにしてレビューしていただこうと考えていました!

@Tksn07 Tksn07 marked this pull request as ready for review December 29, 2023 17:16
@Tksn07 Tksn07 requested a review from a team as a code owner December 29, 2023 17:16
@Tksn07 Tksn07 requested review from y-chan and removed request for a team December 29, 2023 17:16
@Tksn07
Copy link
Contributor Author

Tksn07 commented Dec 29, 2023

@Hiroshiba コンポーネント群の修正完了しました!
レビューお願いします🙏🙏🙏

@Tksn07 Tksn07 requested a review from Hiroshiba December 29, 2023 17:16
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!!!

とても丁寧な進行、ありがとうございます!!!
ぜひまた次もチャレンジいただけるととても助かります🙇
中にはチェックがかなり大変な部分とかもあると思いますので、その際は遠慮なくお聞きいただければ…!!

@Hiroshiba Hiroshiba merged commit fb34081 into VOICEVOX:main Dec 30, 2023
7 checks passed
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