Skip to content

Conversation

@gjaegyun
Copy link
Contributor

💡 배경 및 개요

match date 순으로 정렬 및 match 자세히 보기 클릭 시 date 초기화 이슈 해결

📃 작업내용

match date 순으로 정렬 및 match 자세히 보기 클릭 시 date 초기화 이슈 해결

2025-04-28.8.07.44.mov

@gjaegyun gjaegyun requested review from 976520 and Ethen1264 April 28, 2025 11:08
@gjaegyun gjaegyun self-assigned this Apr 28, 2025
@netlify
Copy link

netlify bot commented Apr 28, 2025

Deploy Preview for gsm-gogov3 ready!

Name Link
🔨 Latest commit 9b6c16b
🔍 Latest deploy log https://app.netlify.com/sites/gsm-gogov3/deploys/680f6d0da4ec1700081b4b87
😎 Deploy Preview https://deploy-preview-220--gsm-gogov3.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@Ethen1264
Copy link
Contributor

기존에 localstorage에 저장을 하게 되면 사용자가 자세히보기를 누른 후 -> 브라우저 종료 -> 다시 접속시 기존에 localstorage에 값이 남아있어 오늘의 날짜가 아닌 전에 선택한 날짜가 지정되어 있을 것 같아요

2025-04-28.8.17.07.mov


localStorage.setItem('matchStatus', JSON.stringify(matchStatusData));
localStorage.setItem('match', JSON.stringify(matchData));
sessionStorage.setItem('selectDate', JSON.stringify(selectDate));
Copy link
Contributor

Choose a reason for hiding this comment

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

이렇게 date만 하면 stage별로 구분이 안 되어서 a stage에서 date를 설정해서 세션에 저장을 하면 b stage에도 동일하게 반영되어 버리는 문제가 생길 것 같아요

Copy link
Contributor Author

Choose a reason for hiding this comment

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

4dc0568

수정하였습니다

@Ethen1264
Copy link
Contributor

2025-04-28.8.25.01.mov

이렇게 date의 항목 위치를 변경하고 뒤로 가기를 하면 날짜에 아무것도 선택되지 않는 문제가 있어요

@Ethen1264
Copy link
Contributor

2025-04-28.8.28.22.mov

현재 날짜의 매치로 들어가면 세션에 ""로 저장이 되어버리는 문제가 있어요

@gjaegyun
Copy link
Contributor Author

이렇게 date의 항목 위치를 변경하고 뒤로 가기를 하면 날짜에 아무것도 선택되지 않는 문제가 있어요

수정했습니다 9ea7715

@gjaegyun
Copy link
Contributor Author

현재 날짜의 매치로 들어가면 세션에 ""로 저장이 되어버리는 문제가 있어요

빈 문자열일 경우 조건식 추가했습니다! 9b6c16b

@gjaegyun gjaegyun merged commit aeaaff2 into develop Apr 28, 2025
5 checks passed
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.

4 participants