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

Network 모듈 이름을 NetworkAPIKit으로 변경 / 네트워크 장애 또는 안정화 시 토스트 알림 / Script 동작 조건 수정 #130

Merged
merged 6 commits into from
Mar 25, 2024

Conversation

WhiteHyun
Copy link
Member

@WhiteHyun WhiteHyun commented Mar 25, 2024

Screenshots 📸

시연 영상
RPReplay_Final1711347735

고민, 과정, 근거 💬

Renamed 📝

  • 네트워크 모니터링을 하기 위해 import Network를 해야했으나, 이미 제가 따로 Network모듈을 만들어놔서 불가피하게 이름을 변경해야 했습니다.

Feature ✨

  • 그리고 Network Monitor를 사용하여 기기의 네트워킹 정보를 전용 큐에서 계속 확인할 수 있도록 구현해두었습니다.

  • 최상단 App에서 onReceive(_:)를 사용해서 데이터가 변경될 때마다 Toast를 띄우도록 했어요.

    그나저나 토스트가 배경색과 비슷하다보니 너무 안 보이네요.. 색상을 변경할까요? 😂😂😂

  • Swiftformat, swiftlint 동작 조건을 수정했습니다. 기존에는 Debug모드일 때 동작하도록 구현해두었는데, Staging이 생겨서 해당 조건식을 삭제하는 게 낫겠더라고요. :)


@WhiteHyun WhiteHyun added 🔧 Enhancement Improvement or Feedback Implementation 🔨 Implementation New feature or request labels Mar 25, 2024
@WhiteHyun WhiteHyun added this to the v2.0.0 milestone Mar 25, 2024
@WhiteHyun WhiteHyun requested a review from eung7 March 25, 2024 06:29
@WhiteHyun WhiteHyun self-assigned this Mar 25, 2024
@WhiteHyun WhiteHyun linked an issue Mar 25, 2024 that may be closed by this pull request
1 task
@WhiteHyun
Copy link
Member Author

WhiteHyun commented Mar 25, 2024

너무 안 보여서 색상을 역전시켰습니다!
훨 낫네요 ㅎㅎ
RPReplay_Final1711351243

@WhiteHyun WhiteHyun changed the title Network 모듈 이름을 NetworkAPIKit으로 변경 / 네트워크 장애 또는 안정화 시 토스트 알림 Network 모듈 이름을 NetworkAPIKit으로 변경 / 네트워크 장애 또는 안정화 시 토스트 알림 / Script 동작 조건 수정 Mar 25, 2024
eung7
eung7 previously approved these changes Mar 25, 2024
Copy link
Contributor

@eung7 eung7 left a comment

Choose a reason for hiding this comment

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

배경 색상을 이번 편행 메인 컬러로 바꿔보는건 어떨까요..?!

@eung7
Copy link
Contributor

eung7 commented Mar 25, 2024

방금 구현하신 것도 마음에 드네요!!! ㅎㅎㅎ

@WhiteHyun
Copy link
Member Author

WhiteHyun commented Mar 25, 2024

편행 메인 색으로 바꿔봤는데 어떠신가요? 저는 방금 바꾼 편행색이 제일 마음에 드는 것 같아요 ㅋㅋ 가독성도 훨 좋고요!
바꿔 둘까요?


RPReplay_Final1711352328

@eung7
Copy link
Contributor

eung7 commented Mar 25, 2024

네! 훨씬 좋은 것 같아요 !! 수고하셨습니다 ㅎㅎ

@WhiteHyun WhiteHyun merged commit 0daf91d into main Mar 25, 2024
2 checks passed
@WhiteHyun WhiteHyun deleted the feature/networking/129 branch March 25, 2024 08:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔧 Enhancement Improvement or Feedback Implementation 🔨 Implementation New feature or request
Projects
Development

Successfully merging this pull request may close these issues.

[Feat]: Network 장애 시 토스트 연결
2 participants