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

ヘルプでアップデートを確認できるようにする #690

Merged

Conversation

sopisoft
Copy link
Contributor

@sopisoft sopisoft commented Feb 6, 2022

内容

ソフトウェアのバージョン と GitHub の Release の最新バージョンを比較して、ヘルプでアップデートを確認します。

関連 Issue

#235

スクリーンショット

最新版を使っているとき

古いバージョンを使っているとき

@sopisoft sopisoft marked this pull request as ready for review February 6, 2022 09:01
@sopisoft sopisoft marked this pull request as draft February 6, 2022 09:03
@sopisoft sopisoft marked this pull request as ready for review February 6, 2022 11: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.

修正をお願いしたい箇所がありました・・・!

(メモです)
内容的に↓とかぶりそうですが、どちらもあって良い機能だなと思うので、どちらもマージできればと思っています

src/components/UpdateInfo.vue Outdated Show resolved Hide resolved
src/components/UpdateInfo.vue Outdated Show resolved Hide resolved
@raa0121
Copy link
Contributor

raa0121 commented Feb 7, 2022

#543 と被る機能になりますね
@Hiroshiba 私の手が遅いせいなんですが、どうしましょう。

@Hiroshiba
Copy link
Member

Hiroshiba commented Feb 7, 2022

@raa0121 上のコメントにある通り、どちらもマージできればと思っています…!
(アプデ通知は一度見たら再度見れないので)

@sopisoft
Copy link
Contributor Author

sopisoft commented Feb 7, 2022

@raa0121
#543 (comment) の機能と被ってしまっているのは私の確認不足でした。
ご迷惑をおかけして申し訳ありません。

@Hiroshiba
Copy link
Member

@Segu-g さん、 @y-chan さん、よかったらレビューお願いできると・・・!

Copy link
Member

@y-chan y-chan left a comment

Choose a reason for hiding this comment

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

LGTMです!
コード的に問題は見受けられませんでした
ただ、我々としては全てのバイナリのビルドが完了し、HPの更新をするまではPre Releaseにし、HPの更新等のリリース作業が完了してからReleaseに変更するという手順に変わる(変えなければならない)点に留意する必要はありそうです。

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!!

リリースは気をつけなきゃですね・・・!
リリース手順書かねば。。。

src/components/UpdateInfo.vue Show resolved Hide resolved
@Hiroshiba
Copy link
Member

マージします・・・!

@Hiroshiba Hiroshiba merged commit 1f9749f into VOICEVOX:main Feb 8, 2022
@Hiroshiba
Copy link
Member

Hiroshiba commented Feb 9, 2022

@gobosan さん
そういえばちょうど、ユーザーさんから要望の多い機能をリストアップしてみました。
参考になりますと幸いです・・・!

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.

4 participants