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

타이머 배경 #93

Open
wants to merge 7 commits into
base: feat/theme-image-preview
Choose a base branch
from

Conversation

anyl92
Copy link
Member

@anyl92 anyl92 commented Dec 29, 2024

💡 다음 이슈를 해결했어요.

  • 타이머 배경 API 연결
  • 삭제 다이얼로그 등 약간의 UI 수정
  • 토스트 UI 및 문구 수정

해결해야 할 이슈

  • 이미지 로딩이 너무 느려서 깜빡임 있음
  • 타이머 라인이 보임

✅ 셀프 체크리스트

  • 브랜치 전략에 맞는 브랜치에 PR을 올리고 있습니다. (master/main이 아닙니다.)
  • 커밋 메세지를 컨벤션에 맞추었습니다.
  • 변경 후 코드는 컴파일러/브라우저 warning/error 가 발생시키지 않습니다.
  • 변경 후 코드는 기존의 테스트를 통과합니다.
  • 테스트 추가가 필요한지 검토해보았고, 필요한 경우 테스트를 추가했습니다.
  • docs 수정이 필요한지 검토해보았고, 필요한 경우 docs를 수정했습니다.

@anyl92 anyl92 requested a review from lgrin-byte December 29, 2024 08:35
@anyl92 anyl92 self-assigned this Dec 29, 2024
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