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

[動画機能] 動画追加・一覧が欲しい #7420

Open
komagata opened this issue Feb 21, 2024 · 34 comments · May be fixed by #7535
Open

[動画機能] 動画追加・一覧が欲しい #7420

komagata opened this issue Feb 21, 2024 · 34 comments · May be fixed by #7535

Comments

@komagata
Copy link
Member

komagata commented Feb 21, 2024

詳細はこちら。

動画機能 · fjordllc/bootcamp Wiki

@komagata komagata changed the title [動画機能] 動画追加が欲しい [動画機能] 動画追加・一覧が欲しい Feb 21, 2024
@a-terumoto-gs
Copy link
Contributor

@komagata @machida
お疲れ様です!
こちらのissueの実装のイメージについてすり合わせをさせてください!
ページとしては一覧ページ、新規追加ページ、動画の詳細ページの3つが必要かと考えております。

全体イメージ

/movies

一覧ページ
左のポートフォリオの下に動画というアイコンを一つ追加してそこからアクセス可能にする
画面のデザインとしてはポートフォリオのものに倣うかたちで、ヘッダー赤枠の中を「動画」のページに合う文言に変更
右上の追加ボタンから新規追加ページにアクセス可能に(/movies/new)
一つ一つの動画をクリックして詳細ページにアクセス可能に(/movies/movie_id)
ポートフォリオ一つ一つが表示されている部分に動画のサムネイル、タイトル、アップロード者、タグなどを表示

moviesイメージ

/movies/new

新規追加ページ
タイトル、説明、動画データ、タグ、公開範囲を入力するフォームを用意
(いろんなページからデザイン拝借して必要なものを並べてみただけのイメージになっているので実装進める中で変わると思います。その際はまたご相談させていただきますので、今のイメージは参考程度にお願いします)
moviesnewイメージ

/movies/movie_id

詳細ページ
このページにアクセスすることで動画を視聴可能にする
タイトル、説明、動画データ、作成者、タグ、公開範囲を表示
(今のイメージはざっくりと必要そうな情報並べてみただけになっておりますのでこちらも参考程度に…)
moviesmovieidイメージ

全体の何となくのイメージとしてはこんな感じでよろしいでしょうか?

細かい仕様

  • 一覧ページの動画一つ一つに表示する情報
    動画のサムネイルを表示した下に、タイトル、アップロード者、タグを表示しようと考えているのですが、不要な情報、または、他に入れた方がいい情報はありますでしょうか?
  • 詳細ページのURLについて
    現在考えているような簡単なもののままでよいのか、user_idと結びつけたものにしたほうがいいのか、悩んでおります。
    どう実装するのがよりベターか決めきれておりません。
    アドバイスいただきたいです。

私自身まだ仕様を詰め切れていないところがあるのと、イメージ図がうまく作成できていない、そしてふわっとしていて申し訳ないのですが、確認よろしくお願いいたしますm(__)m

@komagata
Copy link
Member Author

@a-terumoto-gs

全体の何となくのイメージとしてはこんな感じでよろしいでしょうか?

はい、おっしゃる通りで大丈夫です。ポートフォリオのページを参考にしてるのもその通りでいい感じです〜。

一覧ページの動画一つ一つに表示する情報

おっしゃる通りで大丈夫です〜。

詳細ページのURLについて

おっしゃっている設計でバッチリです〜。/movies/:id

@a-terumoto-gs
Copy link
Contributor

@komagata
コメントありがとうございます。
ではこの方向性で進めさせていただきます!

@a-terumoto-gs a-terumoto-gs linked a pull request Mar 13, 2024 that will close this issue
@machida
Copy link
Member

machida commented Mar 13, 2024

@a-terumoto-gs
https://github.com/fjordllc/bootcamp/wiki/%E5%8B%95%E7%94%BB%E6%A9%9F%E8%83%BD
動画機能のドキュメントを更新しました。
ご確認お願いします。
大きいところは、公開制限を仕様から外したところになります。

@a-terumoto-gs
Copy link
Contributor

@machida
ありがとうございます!
更新された箇所に仕様を合わせて続きを実装したいと思いますm(__)m

@machida
Copy link
Member

machida commented Mar 14, 2024

@a-terumoto-gs よろしくお願いしますー🙏

@a-terumoto-gs
Copy link
Contributor

@komagata @machida
お疲れ様です!
一覧表示で使用する動画のサムネイルの仕様について相談させてください。

簡単にですが調べたところ、FFmpegやImageMagickなどのgemを使えばアップロードした動画から自動生成が可能みたいなので、そういった仕様にしようかなと考えていますが、いかがでしょうか?

または、動画追加の際の手間は増えますが、それ用のフォームを追加してサムネイルもユーザー側で追加をしてもらう仕様にする、というのも案の一つとして考えています。
(上の案がうまく行かなかった際はこの仕様を取り入れるのを候補にいれています)
他に良い案があればご教示いただけると助かります。

確認よろしくお願いいたします。

@komagata
Copy link
Member Author

@a-terumoto-gs bootcampでは最近、動画慣例のライブラリをimagemagickからlibvipsに変えました。
image_processingを通じてサムネ作れると思うのでそちらちょっと調べてみてください〜。

libvipsを活用し『家族アルバム みてね』のサムネイル生成を倍速にしました - mitene / FamilyAlbum Team

@a-terumoto-gs
Copy link
Contributor

@komagata
承知しました!調べてみますm(__)m

@a-terumoto-gs
Copy link
Contributor

メモ
WIP機能については実装を検討したが、編集機能とセットでないと運用ができないと判断し見送ることにした
以下のissueに取り組む際に追加でよいのではないか。
#7421

@a-terumoto-gs
Copy link
Contributor

a-terumoto-gs commented Mar 22, 2024

タグ付け機能がうまく動作しておらず、新規タグ追加の際に反映されないバグあり。
タグの文字を追加後、タグ記入欄でエンターを押して確定させないと起こるようである。
DocsやQ&Aにおいても同様の現象が起こるため、このissueでは一旦そのままの仕様で実装する

Image from Gyazo

@a-terumoto-gs
Copy link
Contributor

@komagata @machida
お疲れ様です!
こちらのissueのについてなのですが、サンプル動画を用意・追加していただく必要があるかなと思っておりますが、いかがでしょうか?
いままで開発環境でのセットアップとテストに必要だったのは画像だけだったと思います。
一応test/fixtures/filesを確認したところ、画像データしかなかったためです。
アップロード形式としてmp4形式とmov形式を想定しているためその二つのデータがあればありがたいです。
よろしくお願いいたしますm(__)m

@komagata
Copy link
Member Author

komagata commented Apr 1, 2024

@a-terumoto-gs 適当な画像や動画(著作権的に問題のないもの)を探して使ってみてください~。

@a-terumoto-gs
Copy link
Contributor

@komagata
承知しました!

@a-terumoto-gs
Copy link
Contributor

a-terumoto-gs commented Apr 19, 2024

@komagata @machida
お疲れ様です!
こちらのissueでサムネ生成に使うために、gemのstreamio-ffmpegを導入しました。

レビューしていただく段階で、そもそも開発環境にFFmpegがインストールされていないとgemを入れても動かず、動作がうまくいかないことが判明しました。
↓レビューしてくださっている@kyokucho1989さんのコメント
#7535 (comment)
(自身は実装の段階で先にFFmpegそのものでのサムネ作成を検討し、インストール済みだったので見落としていました)
これを受けて、開発環境の立ち上げの際にFFmpegをいれる過程の追加が必要と考えました。

libvipsと同様に、READMEにその旨を記載し、wikiページに追記するのがよいかと考えますが、いかがでしょうか?
https://github.com/fjordllc/bootcamp/wiki/%E7%94%BB%E5%83%8F%E5%87%A6%E7%90%86%E3%83%A9%E3%82%A4%E3%83%96%E3%83%A9%E3%83%AA%E3%81%AE%E3%82%A4%E3%83%B3%E3%82%B9%E3%83%88%E3%83%BC%E3%83%AB

他の方法がございましたら、ご教授いただけると助かります。
また、この方向性でよい場合、READMEへの追記とwikiページの編集はこちらで行ってよろしいでしょうか?
確認よろしくお願いいたします。

@machida
Copy link
Member

machida commented Apr 19, 2024

@a-kuroki-gs README にffmpegが必要であることと、そのインストール方法の記入をお願いしたいです🙏
wiki も編集して大丈夫ですー

@komagata
Copy link
Member Author

@a-terumoto-gs (CC: @machida )

なるほどですね。

元々、動画の変換などは外部APIに任せてFFmpegなどはインストールさせないで行きたいと考えていました。動画の変換と一緒にサムネイル作成も別Issue(動画の変換のIssue)にお願いしようと思うのでこちらのIssueではサムネイル作成しなくてよいです。

サムネイル作成と動画の変換は別IssueでGoogle Transcode APIを使ってやりたいと思います。

@machida
Copy link
Member

machida commented May 1, 2024

@a-terumoto-gs 了解です!

@machida
Copy link
Member

machida commented May 11, 2024

@a-terumoto-gs
お待たせしました!
アイコン用意しましたー
https://www.dropbox.com/scl/fi/ft7jwcq2jb78xzosajjud/ghost.png?rlkey=uzotv09fx4clyouy5i36lhxpd&dl=0

@a-terumoto-gs
Copy link
Contributor

@machida
ありがとうございます!

@a-terumoto-gs
Copy link
Contributor

a-terumoto-gs commented May 16, 2024

@machida
お疲れ様です。
追加していただいたアイコンを使用し、ユーザー削除された際の表示を追加しました。
一覧ページのほうは問題ないかと思うのですが、個別ページのほうの表示がうまくいかず…
追加でデザインを入れていただいてもよろしいでしょうか?
(こちらのPRのレビューは完了しており、リリースの予定もまだ立っていないため急ぎではありません)

image

image

お忙しいところ恐縮ですが確認よろしくお願いいたします。

Copy link

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@github-actions github-actions bot added the stale label Jul 29, 2024
Copy link

github-actions bot commented Aug 6, 2024

このissue|PRはstaleラベルを付けた後7日間更新がないためcloseしました。

@github-actions github-actions bot closed this as completed Aug 6, 2024
@a-terumoto-gs a-terumoto-gs reopened this Aug 8, 2024
@komagata komagata moved this to レビュー中 in bootcamp Aug 21, 2024
Copy link

github-actions bot commented Oct 8, 2024

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

Copy link

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@github-actions github-actions bot added the stale label Dec 13, 2024
@komagata komagata removed the stale label Dec 18, 2024
@komagata komagata moved this from レビュー中 to やる in bootcamp Feb 5, 2025
Copy link

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: やる
Development

Successfully merging a pull request may close this issue.

3 participants