Skip to content

Conversation

@HyeonSeok7
Copy link
Contributor

@HyeonSeok7 HyeonSeok7 commented Mar 12, 2025

🤘 작업 내용

  • splash screen 추가 및 적용

📋 변경된 내용

  • splash screen api 추가
  • splash screen 사용될 리소스 아이콘 추가 및 적용

💻 동작 화면

splash

📌 비고

  • 비고 없음

@HyeonSeok7 HyeonSeok7 requested a review from soopeach March 12, 2025 10:03
@HyeonSeok7 HyeonSeok7 changed the title 스플래시 스크린을 추가하고 스타일 지정 스플래시 스크린을 추가하고 스타일 적용 Mar 12, 2025
Copy link
Member

@soopeach soopeach left a comment

Choose a reason for hiding this comment

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

feature/home 은
화면 단위의 모듈인데 스플레시 api를 implementation 및 theme 설정을 하는게 이상한 느낌이에요..! feature/home에 종속된 기능이 아니니까.
다른 곳으로 빼는게 좋지 않을까요?

@HyeonSeok7 HyeonSeok7 requested a review from soopeach April 7, 2025 04:47
@HyeonSeok7
Copy link
Contributor Author

@soopeach 쏘리 너무 늦게봤다.
작업하다 보니 feature/home에 넣은지 몰랐어..ㄷㄷ
너 말대로 우리는 지금 화면별로 feature를 나누고 있으니 app단으로 옮기는건 어떻게 생각해?

@soopeach
Copy link
Member

soopeach commented May 7, 2025

@soopeach 쏘리 너무 늦게봤다. 작업하다 보니 feature/home에 넣은지 몰랐어..ㄷㄷ 너 말대로 우리는 지금 화면별로 feature를 나누고 있으니 app단으로 옮기는건 어떻게 생각해?

좋은 것 같아요!

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