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

feat: add support for heic format #438

Merged

Conversation

ayuki-joto
Copy link
Collaborator

@ayuki-joto ayuki-joto commented Oct 15, 2022

🎩 What? Why?

active storageでの.HEIC拡張子の対応

📌 Related Issues

  • Related to #?
  • Fixes #?

📋 Subtasks

  • Add CHANGELOG upgrade notes, if required
  • If there's a new public field, add it to GraphQL API
  • Add documentation regarding the feature
  • Add/modify seeds
  • Add tests
  • Another subtask

📷 Screenshots (optional)

Description

@takahashim
Copy link
Collaborator

@takahashim
Copy link
Collaborator

@ayuki-joto この修正をローカルの開発環境で試してみると、quillエディタからのダイレクトアップロードに失敗するんですが、何か特殊な設定は必要でしょうか?
システムの管理画面では、organizationに対して「許可されたファイル拡張子」でもろもろheicを許すようにはしているつもりですが、

ActiveRecord::RecordInvalid (バリデーションに失敗しました: Fileファイルは image/jpg, image/jpeg, image/gif, image/png, image/bmp, image/ico, image/heic のうちのどれかを使用してください):

というエラーで失敗するようでした。

imagemagickは ImageMagick 7.1.0-44 を使っています。

@ayuki-joto
Copy link
Collaborator Author

ayuki-joto commented Oct 16, 2022

おっと、quillエディタでHEIC画像をあげた場合ということでしょうか?
ちなみにどのユーザでどの部分でのuploadeでしょうか?
提案コンポーネントで画像をあげる際にエラーが出るというのは既存の問題としてあったりします

@takahashim
Copy link
Collaborator

提案コンポーネントとかブログコンポーネントとかです。
jpg画像では成功しました。

@takahashim
Copy link
Collaborator

もろもろ問題ありそうなところを ayuki-joto#16 でPRしましたが、これでもどうも問題ありそう…? というところです。

@ayuki-joto
Copy link
Collaborator Author

uploadは確認できましたが、HEICの画像はwidth, heightが大きいのでdecidim側のuploadの制限にかかって422が変えることが多そうです。
ただ、uploadされた画像のプレビュー(/rails/active_storage/blob/~のURLで表示されるもの)にがダメそうです。普通の表示(/rails/active_storage/representations/~のURLで表示されるもの)は問題なく表示できてそう

@takahashim
Copy link
Collaborator

試してみましたが、image/heicだけではなくimage/heifも許すようシステム管理画面に追加するとエラーが出なくなるようでした。
とはいえChromeでは表示されないようなので、ブラウザの画面上では壊れた画像になります。

@ayuki-joto ayuki-joto merged commit 13f3c80 into codeforjapan:develop Oct 24, 2022
@ayuki-joto ayuki-joto mentioned this pull request Oct 25, 2022
6 tasks
@ayuki-joto ayuki-joto deleted the feature/add-support-heic-format branch May 24, 2024 08:56
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.

2 participants