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] 열람실 이용관련 Notification 처리 #69

Merged
merged 13 commits into from
Nov 10, 2024

Conversation

Seokki-Kwon
Copy link
Contributor

@Seokki-Kwon Seokki-Kwon commented Nov 9, 2024

AS-IS

열람실 이용관련 Notification 처리

TO-BE

열람실 이용관련 Notification 처리

KEY-POINT

열람실이용 종료 30, 10 분전 Notification 작업을 진행하고, 동시에 백그라운드에서 타이머가 동작하지 않는문제도 같이 수정했습니다.

  • 열람실이용 종료/연장 시 시간 30, 10분전 Notification 등록
  • 백그라운드 진입시 정상적으로 타이머에 시간을 반영하기 위해서 백그라운드 진입시간을 체크하고 재계산하는 방식을 이용했습니다.
  • Background/Foreground 상태를 감지를 위해서 별도의 이벤트핸들러를 등록하고 앱에 상태를 저장하도록 구성했습니다.
  • 열람실 이용종료시 DispatchQueue 대신 Combine으로 지연을 발생시킨뒤 데이터를 다시 불러오도록 변경했습니다.

SCREENSHOT (Optional)

@Seokki-Kwon Seokki-Kwon added the 🔥 Feat 새로운 기능 관련 label Nov 9, 2024
@Seokki-Kwon Seokki-Kwon self-assigned this Nov 9, 2024
Copy link
Contributor

@owllwo98 owllwo98 left a comment

Choose a reason for hiding this comment

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

appState 부분 외에는 다 이해됐습니다!! 고생하셨어요

HongikYeolgong2/Injected/AppState.swift Show resolved Hide resolved
Copy link
Contributor

@Choe-ju Choe-ju left a comment

Choose a reason for hiding this comment

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

확인했습니다 고생하셨습니다!

@Seokki-Kwon Seokki-Kwon merged commit 829f72b into develop Nov 10, 2024
@Seokki-Kwon Seokki-Kwon deleted the Feature/#66-PushNotification branch November 22, 2024 07:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔥 Feat 새로운 기능 관련
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants