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

100MB以上の広告データがアップロードできない問題を修正 #877

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

TkymHrt
Copy link
Collaborator

@TkymHrt TkymHrt commented Oct 1, 2024

対応Issue

resolve #876

概要

  • データを直接minioへ送信するのではなく,クライアント側でデータを分割し,サーバーで結合してからminioへ送信するように修正しました.

  • Loadingコンポーネントに進捗属性を追加

  • 1GB以上のファイルを防ぐバリデーションを追加

画面スクリーンショット等

image
image

テスト項目

  • Cloudflare Tunnel を経由してローカル環境をビルドし,公開する.
  • http://localhost:3000/sponsoractivities にアクセスできること.
  • 広告データとして100MB以上のファイルを追加できること.
    (検証用に使った画像のサイト:https://sample-img.lb-product.com/)
  • 送信中に%が増えていくこと.
  • 画像のプレビューに表示され,minio上で追加したファイルが確認できること.
  • ローディングの表示が確認できる.
  • 1GB以上のファイルを送信できない

備考

Zeenの記事のようにしなくても解決できました!
アップロードするまで,ちょっと遅いかも...
FinanSuのCloudflare Tunnelを利用するためのjsonファイルとcert.pemファイルが見つからなかったため,https://bingo.nutfes.net/で検証しました.確認方法やFinanSuのを知りたいです.
(普通にsettingsに入ってた…)

@TkymHrt TkymHrt requested a review from Kubosaka October 1, 2024 17:01
@TkymHrt TkymHrt self-assigned this Oct 1, 2024
Copy link
Collaborator

@Kubosaka Kubosaka left a comment

Choose a reason for hiding this comment

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

いい感じです!
ログ消してください
少しコメントしました

@TkymHrt TkymHrt requested a review from Kubosaka October 12, 2024 07:30
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.

100MB以上の広告データがアップロードできない問題の修正
2 participants