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

feat: api 기초 구현 & 환경변수 및 스플래시 화면 설정 #3

Merged
merged 8 commits into from
Oct 5, 2024

Conversation

gw-lim
Copy link
Contributor

@gw-lim gw-lim commented Oct 3, 2024

✏️ 작업 내용

  • api instance를 생성했습니다.
  • auth instance에 사용할 커스텀 훅을 간단하게 구현해두었습니다.
  • 환경변수 관련 설정을 해두었습니다.
  • 스플래시 화면을 구현했습니다.

📷 스크린샷

스크린샷 2024-10-03 오후 5 42 18 스크린샷 2024-10-03 오후 5 25 44

🎸 기타

@gw-lim gw-lim added the ✨ feat label Oct 3, 2024
@gw-lim gw-lim self-assigned this Oct 3, 2024
Copy link
Contributor Author

Choose a reason for hiding this comment

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

auth instance 관련 커스텀 훅입니다. 추후에 토큰 관리 방식에 따라서 수정이 필요합니다!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

일반 instance와 auth instance로 분리해두었습니다

Copy link
Contributor

Choose a reason for hiding this comment

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

혹시 png말고 svg로도 가능한가요? 나중에 확장성 고려해서 svg로 해두는것도 가능하다면 좋을 것 같아서요!
너무 많이 바꾸셔야하면 무시하셔도 됩니다

Copy link
Contributor Author

Choose a reason for hiding this comment

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

앗 그럼 일단 png로 해두겠습니다,,

Copy link
Contributor

Choose a reason for hiding this comment

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

splash screen 으로 1s는 약간 짧지 않을까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

스플래시 스크린은 이후에 토큰을 확인하는 시간 동안 띄워둬여할 것 같아 나중에 토큰 로직 합쳐지면 수정해보겠습니다!

Copy link
Contributor

@wjddnjs7895 wjddnjs7895 left a comment

Choose a reason for hiding this comment

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

큰 코멘트는 남기지 않았습니다! 바로 머지하셔도 괜찮을 것 같습니다 ")

@gw-lim gw-lim merged commit 293442f into DevKor-github:develop Oct 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants