PNG파일의 섬네일 생성시 배경이 투명할 수 있도록 수정#2228
Closed
lalafellsleep wants to merge 4 commits into
Closed
Conversation
Contributor
Author
|
파일 업로드에 들어간 설정은 들어갈 위치가 마땅치 않아 임시적으로 넣어두었습니다. 해당 설정을 무시하고 서드파티에서도 컨트롤 할 수 있지만 이러한 설정의 전역 설정(?)이 없는것이 어색할 수 있다고 생각하여 추가하였으나 file모듈의 설정부분이 불필요하다고 판단되는 경우 제외하셔도 무방합니다. |
|
말씀하신 것처럼 allow_thumbnail_transparent 설정을 사용하는 곳이 없는 상태이고, |
Contributor
Author
|
@bnu 확인했습니다. 우선 해당 사항을 분리하고 회원 모듈의 프로필 업로드 시 배경 투명 허용 옵션으로 적용하여 별도 pr 하도록 하겠습니다. |
6256a18 to
0b23124
Compare
|
#2243 PR에서 진행 |
This pull request was closed.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
서드파티 및 Member모듈의 insertProfileImage에서 사용할 수 있도록 확장
기본값: N