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

Fix: LetterBoxDetailLetterView 내 Letter 이슈 및 네이밍 #205

Merged
merged 4 commits into from
Sep 22, 2024

Conversation

uunwon
Copy link
Collaborator

@uunwon uunwon commented Sep 13, 2024

📕 Issue Number

Close #204

📙 작업 내역

구현 내용 및 작업 했던 내역

  • LetterBoxDetailLetterView 내 Letter 를 Published 로 설정해놓아서, 보던 도중 편지가 바뀌는 이슈 해결
  • 직관적이지 않는 파일 네이밍 수정
  • 타겟 iPhone 외 삭제

📘 작업 유형

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

📋 체크리스트

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

📝 PR 특이 사항

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

  • 특이 사항
    계정 이슈가 아직 해결되지 않은 참에 0.5.2 버전에서 발생한 오류와 함께 리팩토링 (== 네이밍 수정) 진행했습니다. 이전에 적절하다고 생각한 파일명이 뒤늦게 복잡하다고 생각되어, 심플하게 변경했습니다. 그리고 이전 0.5.4 빌드하면서 타겟 삭제하는 작업을 진행했는데요. 공통으로 해야 하는 작업이라고 생각되어서 요번에 삭제해서 올립니다.



@uunwon uunwon self-assigned this Sep 13, 2024
@YuleGlycerine YuleGlycerine merged commit 2a23c85 into Develop Sep 22, 2024
1 check passed
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.

[LetterBox] LetterBoxDetailLetterView 내 Letter 이슈
4 participants