-
Notifications
You must be signed in to change notification settings - Fork 312
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
複数エンジン対応:アイコン情報をengine manifestから取得するように #953
複数エンジン対応:アイコン情報をengine manifestから取得するように #953
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!
いつもありがとうございます!!
@@ -473,7 +479,7 @@ export default defineComponent({ | |||
]; | |||
} | |||
} | |||
watch(engineInfos, updateEngines, { immediate: true }); // engineInfosを見て動的に更新できるようにする | |||
watch([engineInfos, engineManifests], updateEngines, { immediate: true }); // engineInfos、engineManifestsを見て動的に更新できるようにする |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@raa0121 さん、もしよかったらレビューお願いできると心強いです・・・! 🙏 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
問題ないと思います。
5340aa5
to
cf6bf19
Compare
rebase mainしましたー。conflict無くマージできるはずです |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
大丈夫そうなのでマージします!!ありがとうございました!!
@raa0121 さんもレビューありがとうございます! |
内容
MenuBarやAudioCellのアイコンをengine manifestから取得するようにします。
関連 Issue
スクリーンショット・動画など
正常に動作しているのが確認出来ました。


その他
(なし)