Skip to content

Comments

Feat/import photo#285

Merged
ksiomng merged 21 commits intoDevelopfrom
Feat/ImportPhoto
Jan 7, 2025
Merged

Feat/import photo#285
ksiomng merged 21 commits intoDevelopfrom
Feat/ImportPhoto

Conversation

@ksiomng
Copy link
Collaborator

@ksiomng ksiomng commented Jan 6, 2025

📕 Issue Number

Close #269

📙 작업 내역

구현 내용 및 작업 했던 내역

  • 편지 쓰기뷰 이미지 미리보기
  • 편지 쓰기뷰 이미지 프레임
  • 파일 정리

📘 작업 유형

  • 신규 기능 추가
  • 버그 수정
  • 리펙토링
  • 문서 업데이트

📋 체크리스트

  • Merge 하는 브랜치가 올바른가?
  • 코딩컨벤션을 준수하는가?
  • PR과 관련없는 변경사항이 없는가?
  • 내 코드에 대한 자기 검토가 되었는가?
  • 변경사항이 효과적이거나 동작이 작동한다는 것을 보증하는 테스트를 추가하였는가?

📝 PR 특이 사항

PR을 볼 때 주의깊게 봐야하거나 말하고 싶은 점

  • 작업된 뷰를 정리하다가 그냥 모든파일을 정리해서 파일변경이 많습니다.
  • 이번에 작업된 내용은 ContentWriteView 에서만 확인하면 될거같습니다 ...
  • 디자인이나 코드 잘못된 부분을 말씀해주시면 수정하고 머지할게요 !!!



Copy link
Collaborator

@the-hye the-hye left a comment

Choose a reason for hiding this comment

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

오우 대규모 변경,, 박수 쨕쨕👏

@YuleGlycerine
Copy link
Collaborator

와아아앙 이거 빨리 테플 올려보고싶네요!! 디자인 조아요!. 그런데.. 그림자가 안들어간 것을 지금 보았읍니다.. 일단 머지하구 그림자는 이후에 넣을까나요? 요 기능 자체를 넘 기다렸어요. 이것만 메인에 합쳐서 업데이트 올리는 것도 생각해봄직 하네요..

@ksiomng ksiomng merged commit c7c5ec8 into Develop Jan 7, 2025
1 check passed
@ksiomng ksiomng self-assigned this Jan 27, 2025
@ksiomng ksiomng deleted the Feat/ImportPhoto branch February 10, 2025 01:12
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.

[03-04] 동봉된 사진 미리보기

3 participants