-
Notifications
You must be signed in to change notification settings - Fork 3
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/227 위치정보를 불러올 수 없는 경우 위치정보가 필요한 기능들을 막는다. #255
base: develop
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다!🥰
현재 상태는 최초에 위치값이 없으면 새로고침하지 않는 한 위치를 받아올 수 있게 되더라도 글은 계속 쓸 수 없게 되는데요.
순간의 오류로 위치를 받아오지 못할 때도 무조건 새로고침을 해야하는 상황이네요.
버튼을 구현하고 버튼을 누를 때마다 위치를 검사하는건 어떨까...하는 생각도 듭니다.🤔
또 PC로 사용하는 경우 위치정보의 유무와 상관 없이 모바일을 권장하는 편이 낫지 않을까요?
위치와 실행 환경은 완전 별도의 이슈가 아닐까..하는 생각이 듭니다.
마지막으로 위치정보가 필요한 기능
에는 댓글 작성도 있는데요.
댓글 작성 창도 지금의 버튼처럼 없애버리는 건 어떨까요?
@@ -15,7 +15,10 @@ | |||
목록 | |||
</router-link> | |||
<MapAssistantButtons v-show="isDefaultMode" /> | |||
<PostCreateButton class="post-create-btn" /> | |||
<PostCreateButton | |||
v-show="isLocationIdentifiable" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
최초에 한번만 isLocationIdentifiable
을 검사하는데요.
그렇다면 v-if
가 더 맞을 것 같아요.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
우 안녕하세요! 코멘트들 다 꼼꼼하게 달아주신게 느껴지네요 감사합니다 🥰
리뷰 반영하다 반영하다 이 코멘트가 마지막 반영할 코멘트여서 이 댓글로 전체적인 의견 남길게요.
현재 상태는 최초에 위치값이 없으면 새로고침하지 않는 한 위치를 받아올 수 있게 되더라도 글은 계속 쓸 수 없게 되는데요.
순간의 오류로 위치를 받아오지 못할 때도 무조건 새로고침을 해야하는 상황이네요.
버튼을 구현하고 버튼을 누를 때마다 위치를 검사하는건 어떨까...하는 생각도 듭니다.🤔
이 코멘트에 대해서는 현재 위치
버튼을 누르면 위치 정보가 생겨날 경우 글쓰기 버튼이 나타나도록 구현했습니다!
새로고침을 안 해도 되어요. 그래서 글쓰기 버튼의 v-show
속성은 그대로 놔두겠습니다. (런타임 시 바뀔 여지가 있기 때문이에요)
또 PC로 사용하는 경우 위치정보의 유무와 상관 없이 모바일을 권장하는 편이 낫지 않을까요?
위치와 실행 환경은 완전 별도의 이슈가 아닐까..하는 생각이 듭니다.
이 코멘트에 대해서는 팀원들과 얘기를 나눠봤는데 PC에서 위치를 받아올 수 있는 경우 글쓰기도 댓글쓰기도 가능하므로 굳이 모바일 권장을 하지 않아도 된다는 쪽으로 의견이 모였는데 어떻게 생각하시나요?
처음에 위치와 실행환경을 고려한 이유는 (위치를 받아올 수 없는 경우 실행환경에 따라 분기를 친 이유는)
- PC인 경우 위치값을 받아오지 못하는 이유가 PC이기 때문임이 거의 명확하고 -> 따라서 모바일 환경을 권장해줘야 한다고 생각했고
- 모바일인 경우 높은 확률로 위치값을 받아올 수 있기 때문에, 위치값을 받아오지 못하는 경우 그 사실만을 명시해줘도 충분하다고 생각했기 때문이에요.
엘리의 코멘트에 대한 답글 달면서 생각해봤는데 모바일 환경에 따른 분기를 나중에 쳐줄 수도 있겠네요!
마지막으로 댓글 창도 우의 의견 받아서 위치값을 받아올 수 없는 경우 없애버렸습니다!
이 경우에 새로 위치를 받아올 수 있는 여지가 없기 때문에 v-if
로 설정해주었습니다.
늘 꼼꼼한 리뷰 멋져요 우 💯
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고했어요 예지 👍
몇 가지 피드백 남겼으니 확인 부탁해요!
@@ -1,5 +1,7 @@ | |||
const ERROR_MESSAGE = { | |||
UNIDENTIFIABLE_LOCATION: "😢 위치 정보를 확인할 수 없습니다.", | |||
UNIDENTIFIABLE_LOCATION_PC: "😢 위치 정보를 불러올 수 없습니다. 모바일 환경 사용을 권장합니다.", | |||
UNIDENTIFIABLE_LOCATION_MOBILE: "😢 위치 정보를 불러올 수 없습니다. 둘러보기만 가능합니다.", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
둘러보기만 가능합니다.
보다는 어떠한 액션을 취하면 제대로 사용할 수 있는지 말해주는 게 좋을 것 같아요!
설정 - Chrome - 위치 정보 수집 허용
을 선택해주세요. 이런 식으로 알림창을 띄워주는 방법도 있겠네요!
다만 이 부분은 모바일 환경에서도 안드로이드 iOS 이렇게 분기가 생기겠네요... 🤔
새로운 이슈로 파서 작업하고 배포를 한 번에 진행하는 것도 방법인 것 같아요! 🙂
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
위치 정보를 수집하지 못하는 경우가 여러 케이스가 있을 수 있을 것 같은데요.
(순간적으로 받아오지 못하는 경우라든지..)
위치 정보 수집을 허용해주세요. 라는 말로 묶어버리는게 맞는가 싶네요.
엘리 말대로 새로운 이슈를 파서 진행해도 좋을 것 같아요! 🙂
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨어요 예지니어스!
Resolves #227
사용자의 위치를 받아올 수 없는 경우
그리고 위치정보가 필요한 기능들에서
글쓰기 버튼
을 안 보이게 해줬어요.원래 로그인이 안 된 경우도 맡아서 하려고 했는데 범위가 축소됐네요.
로그인이 안 된 경우 redirect 까지 해서 새로운 이슈를 파야 될 것 같아요.