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

Feature/letter box detail letter view #100

Merged
merged 7 commits into from
Sep 3, 2024

Conversation

uunwon
Copy link
Collaborator

@uunwon uunwon commented Sep 3, 2024

📕 Issue Number

Close #

📙 작업 내역

구현 내용 및 작업 했던 내역

  • 날짜 검색시, 시간 제한받지 않기
  • 키보드 자동 Capslock 끄기
  • 안 읽음 상태 업데이트를 위한 화면 갱신

  • ToastView padding 고정값 부여
  • 편지 내용 빈 배열일시, 빈 편지지 뜨도록
  • 검색 키워드 소문자로 내보내기

📘 작업 유형

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

📋 체크리스트

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

📝 PR 특이 사항

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

  • 특이 사항
    어제 에뮬레이터에 직접 돌려보면서 생긴 사소한 오류들 정돈했습니다.



@uunwon uunwon self-assigned this Sep 3, 2024
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.

lowercased 확인 !

Copy link
Collaborator

@ksiomng ksiomng left a comment

Choose a reason for hiding this comment

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

드가자 ~ . ~

@uunwon uunwon merged commit 14f5abe into Develop Sep 3, 2024
1 check passed
@uunwon uunwon deleted the Feature/LetterBoxDetailLetterView branch November 5, 2024 01:52
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.

3 participants