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

[FE][Feat] #286 : 데스크탑 접속시 모바일 접속 유도 #287

Merged
merged 3 commits into from
Nov 27, 2024

Conversation

happyhyep
Copy link
Member

📝 PR 개요

데스크탑 접속시 모바일 접속 유도


🔍 변경 사항

  • 데스크탑 접속시 모바일 접속 유도

✅ 체크리스트 (Checklist)

  • 코드가 빌드 오류 없이 잘 작동하는지 확인
  • 테스트가 통과하는지 확인
  • 스타일 가이드와 일관성을 유지했는지 확인
  • 관련 문서가 업데이트되었는지 확인 (선택 사항)
  • 리뷰어가 이해할 수 있도록 주석이나 설명을 추가했는지 확인

설명: 이 체크리스트는 PR 작성자가 스스로 점검할 수 있는 기준을 제시합니다. 코드가 빌드 오류 없이 작동하고, 테스트를 통과하며, 스타일 가이드를 따르고 있는지 확인하는 항목들입니다.


🔄 관련 이슈 (Linked Issues)

#286


📷 스크린샷 및 동영상 (선택 사항)

image
image


- 데스크탑 접속시 모바일 접속 유도
@happyhyep happyhyep added the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Nov 26, 2024
@happyhyep happyhyep self-assigned this Nov 26, 2024
@happyhyep happyhyep linked an issue Nov 26, 2024 that may be closed by this pull request
@juwon5272
Copy link
Collaborator

오 이렇게 모바일인지 확인할 수 있군요! 하나 배워갑니다

@juwon5272 juwon5272 self-requested a review November 26, 2024 15:36
juwon5272
juwon5272 previously approved these changes Nov 26, 2024
effozen
effozen previously approved these changes Nov 26, 2024
Copy link
Collaborator

@effozen effozen left a comment

Choose a reason for hiding this comment

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

모바일 접속 유도는 생각도 못했는데 보법이 다르시군요...!

그리고 이렇게 구현할 수 있다는 것도 배워갑니다.

경험이 정말 많으시다는걸 항상 느끼는데.. 그런 점들이 하나하나 녹아나는 모멘트들에서 정말 크게 배웁니다..!

매번 감사드립니다. 😁 👍

frontend/src/App.tsx Show resolved Hide resolved
frontend/src/App.css Outdated Show resolved Hide resolved
frontend/src/App.css Show resolved Hide resolved
frontend/src/App.tsx Show resolved Hide resolved
@happyhyep
Copy link
Member Author

오 이렇게 모바일인지 확인할 수 있군요! 하나 배워갑니다

하핫 모바일인지 아닌지보다는... 그냥 저희 데스크탑 서비스 구현되기 전까지 임시로 차단해둔 정도입니다....

@happyhyep happyhyep dismissed stale reviews from effozen and juwon5272 via c5ba70e November 27, 2024 02:56
Copy link
Collaborator

@leedongyull leedongyull left a comment

Choose a reason for hiding this comment

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

우와 구현된거보니 신기하네요!

@happyhyep happyhyep merged commit f616c77 into frontend Nov 27, 2024
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

데스크탑으로 접속시 모바일로 접속 유도
4 participants