Skip to content

feat: [downloader] 利用規約の表示を改善する#1006

Merged
qryxip merged 5 commits intoVOICEVOX:mainfrom
qryxip:pr/feat-downloader-improve-about-terms
Feb 15, 2025
Merged

feat: [downloader] 利用規約の表示を改善する#1006
qryxip merged 5 commits intoVOICEVOX:mainfrom
qryxip:pr/feat-downloader-improve-about-terms

Conversation

@qryxip
Copy link
Copy Markdown
Member

@qryxip qryxip commented Feb 13, 2025 •
edited
Loading

内容

  1. VVMとVOICEVOX ONNX Runtimeの利用規約をbatでくっつけて表示する。
  2. 表示の順番はVVMの後にVOICEVOX ONNX Runtimeとする。
  3. ターミナル一画面に収まらない場合、minusによるページング表示を行う。それに伴いyとnに加えてrを追加。rを選ぶともう一度ページングが始まる。
  4. yとn、および上記rについて説明を追加。

Image

関連 Issue

Resolves: #1000

その他

@qryxip qryxip requested a review from Hiroshiba February 13, 2025 21:08
@qryxip qryxip mentioned this pull request Feb 13, 2025
Closed
@qryxip qryxip force-pushed the pr/feat-downloader-improve-about-terms branch from bc4f988 to b0904f4 Compare February 13, 2025 21:12
qryxip
qryxip commented Feb 13, 2025
View reviewed changes
Comment thread Cargo.lock
Copy link
Copy Markdown
Member Author

@qryxip qryxip Feb 13, 2025

Choose a reason for hiding this comment

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

そろそろ総クレート数が4桁いきそう…

Hiroshiba
Hiroshiba reviewed Feb 15, 2025
View reviewed changes
Copy link
Copy Markdown
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.

良い感じだと思います！！

ちょっと相談です！！
ページングの一番上の行でこんな感じの案内するのどうでしょう？

ダウンロードには以下の利用規約への同意が必要です。
（矢印キーで移動、q で終了）

なんで利用規約が表示されてるのかの意図を書くのと、あとページャーの簡単な使い方を案内しとくのどうかな～と！

Comment thread crates/downloader/src/main.rs Outdated
Comment thread crates/downloader/src/main.rs Outdated
Comment thread crates/downloader/src/main.rs Outdated
@qryxip
Copy link
Copy Markdown
Member Author

qryxip commented Feb 15, 2025

ちょっと相談です！！
ページングの一番上の行でこんな感じの案内するのどうでしょう？

845d09e (#1006)
操作方法についてはページャーの"prompt"に入れていることから、「同意が必要です」だけ入れてみました。
(ちなみにpromptに複数行の文字列を入れることも考えたのですが、拒否されました)

ターミナルに十分な縦幅があることによりページングが行われなかった場合:

image

ページングが行われた場合:

image

@Hiroshiba
Copy link
Copy Markdown
Member

Hiroshiba commented Feb 15, 2025

一番下のとこ、VSCodeだと真っ黒なんですよねー
image

Git Bashだと描画されてるけど、すごい見づらそう
image

ここは時間割くべき場所じゃないので、もう雑に２行目に括弧書きでとりあえず良いかなぁと。

webページで利用規約用意してあげればもうちょいスマートな方法がありそうだなーと思ってます。
最初に規約の表示方法を選んでもらって（webで開く・ターミナルに表示する・ページャーで開く等）、そのあと同意するか聞けば良いかもな～みたいな。

webページ用意は時間かかるのでとりあえず完成優先が良さそう。
webページは無いけど表示方法聞く→同意するか聞く、の流れにするのはありかも。どちらでも！

Hiroshiba
Hiroshiba approved these changes Feb 15, 2025
View reviewed changes
Copy link
Copy Markdown
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！！

とりあえず現状＋２行目に操作方法書く形であればLGTM！
変更するときはrereviewしていただけると！！（見てみたい）

@qryxip
Copy link
Copy Markdown
Member Author

qryxip commented Feb 15, 2025

ああなるほど、そうなるんですねぇ…
(よくよく考えてみればこういうのはカラースキームありきで、minusにはそういうものは無さそう)

というわけで: 51d922f (#1006)

@qryxip qryxip requested a review from Hiroshiba February 15, 2025 18:19
@Hiroshiba Hiroshiba mentioned this pull request Feb 15, 2025
Closed
Hiroshiba
Hiroshiba approved these changes Feb 15, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@Hiroshiba Hiroshiba left a comment •
edited
Loading

Choose a reason for hiding this comment

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

LGTM！！

0.16はこれでいけそう！

qで閉じたあと、規約が消えた状態で規約に同意しないといけないのはちょっとUX的に微妙なので、新しくissue建てようと思います！

@qryxip qryxip merged commit 9c15c40 into VOICEVOX:main Feb 15, 2025
@qryxip qryxip deleted the pr/feat-downloader-improve-about-terms branch February 15, 2025 18:56
@qryxip qryxip mentioned this pull request Feb 15, 2025
Closed
3 tasks
qryxip added a commit to qryxip/voicevox_core that referenced this pull request Feb 15, 2025
@qryxip
ci: fix up VOICEVOX#1006: deny.tomlの対応
737d6ab
@qryxip qryxip mentioned this pull request Feb 15, 2025
Closed
qryxip added a commit to qryxip/voicevox_core that referenced this pull request Feb 15, 2025
@qryxip
ci: fix up VOICEVOX#1006: deny.tomlの対応
7755468
qryxip added a commit to qryxip/voicevox_core that referenced this pull request Feb 15, 2025
@qryxip
ci: fix up VOICEVOX#1006: deny.tomlの対応
f8c4341
qryxip added a commit to qryxip/voicevox_core that referenced this pull request Feb 15, 2025
@qryxip
ci: fix up VOICEVOX#1006: deny.tomlの対応
a8bd2cc
qryxip added a commit to qryxip/voicevox_core that referenced this pull request Feb 15, 2025
@qryxip
ci: fix up VOICEVOX#1006: deny.tomlの対応
5671fac
qryxip added a commit to qryxip/voicevox_core that referenced this pull request Feb 15, 2025
@qryxip
ci: fix up VOICEVOX#1006: deny.tomlの対応
7694ff3
qryxip added a commit to qryxip/voicevox_core that referenced this pull request Feb 16, 2025
@qryxip
feat: [downloader] fix up VOICEVOX#1006: bat抜きで利用規約を表示する
fd8cb10
@qryxip qryxip mentioned this pull request Feb 16, 2025
Merged
qryxip added a commit to qryxip/voicevox_core that referenced this pull request Feb 16, 2025
@qryxip
feat: [downloader] fix up VOICEVOX#1006: bat抜きで利用規約を表示する
7113211
qryxip added a commit to qryxip/voicevox_core that referenced this pull request Feb 16, 2025
@qryxip
feat: [downloader] fix up VOICEVOX#1006: bat抜きで利用規約を表示する
a9276fb
qryxip added a commit that referenced this pull request Feb 16, 2025
@qryxip
feat: [downloader] fix up #1006: batを使わず利用規約を表示する (#1011)
60790a1 
`bat`を使わずに利用規約をまとめて表示する。ページャーである`minus`はその
まま。表示のしかたは`bat`と違い簡素なものとした。理由としてはあまり手間
をかけられないというのもあるのだが、そもそも`bat`の表示も理想とはちょっ
と違っていたため。

また、 #1006 で更新し忘れたdeny.tomlの他の部分も更新してしまう。

Resolves: #1009
Closes: #1010
@qryxip qryxip mentioned this pull request Feb 16, 2025
Merged
qryxip added a commit that referenced this pull request Mar 13, 2025
@qryxip @Hiroshiba
ci: Check release target tuplesワークフローを追加 (#1012)
cc84dde 
#1006 の副産物。deny.tomlの`graph.targets`をチェックする。

Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Hiroshiba Hiroshiba Hiroshiba approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

2 participants

@qryxip @Hiroshiba