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] 백그라운드에서도 타이머가 돌아갈 수 있도록 #450

Merged
merged 4 commits into from
Feb 2, 2024

Conversation

sumi-0011
Copy link
Member

🤔 해결하려는 문제가 무엇인가요?

  • 앱이 아닌 웹뷰에서 실행하다 보니 백그라운드에서 타이머가 잘 돌아가지 않는 문제가 있었어요
  • 백그라운드에서도 타이머가 돌아갈 수 있도록 수정하였습니다.

@sumi-0011 sumi-0011 self-assigned this Feb 1, 2024
Copy link

vercel bot commented Feb 1, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
10mm-client-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 2, 2024 5:22am

Copy link
Contributor

github-actions bot commented Feb 1, 2024


localStorage.setItem(
STORAGE_KEY.PROGRESS_MISSION.TIME_STACK(missionId),
JSON.stringify([...prevStackData, timeInfo]),
Copy link
Collaborator

Choose a reason for hiding this comment

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

미션마다 time을 스택으로 관리하는 걸까요 스택으로 관리하는 이유가 궁금해요!

const getProgressMissionTimeToStack = (missionId: string) => {
const timeStack = localStorage.getItem(STORAGE_KEY.PROGRESS_MISSION.TIME_STACK(missionId)) || '[]';
const timeStackData = JSON.parse(timeStack);
console.log('timeStackData: ', timeStackData);
Copy link
Collaborator

Choose a reason for hiding this comment

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

나중에 지워주세욤

@sumi-0011 sumi-0011 linked an issue Feb 2, 2024 that may be closed by this pull request
@sumi-0011 sumi-0011 merged commit 49f95a6 into develop Feb 2, 2024
2 of 3 checks passed
@sumi-0011 sumi-0011 deleted the feat/background-timer branch February 2, 2024 05:22
Copy link
Contributor

github-actions bot commented Feb 2, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] 스톱워치 버그사항 수정
2 participants