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

fix: 크로스 브라우징 이슈 해결을 위한 명확한 날짜 포맷 사용 #770

Merged
merged 1 commit into from
Jun 23, 2024

Conversation

R1mmm
Copy link
Member

@R1mmm R1mmm commented Jun 12, 2024

🚩 관련 이슈

image image

📋 작업 내용

  • Date객체로 인한 크로스 브라우징 이슈 해결

📌 PR Point

  • Invalid Date오류였고, 크롬이 아닌 다른 브라우저에서 이슈가 발견된 걸로 보아 크로스 브라우징 이슈일 것이라고 생각했습니다. 검색해보니 이런 글이 있더라구요. Date 객체 사용 시 주의해야 할 점 (크로스 브라우징 이슈) 몇가지 데이트 포맷은 크롬이 아닌 다른 브라우저에서는 제대로 작동하지 않고 Invalid Date로 판명될 수 있으니, 공통되는 포맷을 사용해야 한다는 내용의 글입니다..! 해당 글 참고해서 모든 브라우저에서 사용될 수 있게끔 toDate() 메소드를 쓸 때 포맷을 지정해주니 이슈가 해결된걸로 보여요. 하지만 qa는 한 번 더 진행해봐야 할 것 같습니다... 🥺

📸 스크린샷

@R1mmm R1mmm self-assigned this Jun 12, 2024
@R1mmm R1mmm requested a review from borimong as a code owner June 12, 2024 12:56
Copy link

height bot commented Jun 12, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

@borimong
Copy link
Contributor

확인했습니다!! dayjs 에서는 별다른 파라미터 없으면 디폴트로 new Date 를 반환해서, 그거로 인해 사파리에서 이슈가 생긴 것 같아요! ISO 8601 형식이거나 이렇게 라이브러리에서 직접 파싱해서 써주면 문제가 없을 것 같습니다!! 수고 하셨어요 :)

@R1mmm R1mmm merged commit 38748af into develop Jun 23, 2024
2 checks passed
@borimong borimong deleted the feat/#769 branch July 22, 2024 14:57
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