-
Notifications
You must be signed in to change notification settings - Fork 6
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: 진행 상태 변경시 연락요청 메세지 추가 #267
Conversation
읽다보니까 조금 어색한 감이 있어서, 진행하기 쪽에서 멘트를 조금 바꿔보면 좋을 것 같아요.
이렇게 나눠서 생각좀 해봐야할듯..!! |
씁 그렇네요... 글이 많아지는 걸 고려해서 문장의 순서를 바꾸는 건 어떨까요 멘토에게 연락을 취해주세요. 조금 어색할까요..? |
조금 길어져도 괜찮을 것 같은데.. 이런건 어떨까요?
진행하기 상태로 변경되었습니다. <- 이거 그냥 alert로라도 보여주면 이상할까요? 상태가 변경되었음을 알려야하지 않을까 생각이 드네요. 그리고 모달측은 " 뭐 이런식으로 하면 좋을 것 같아요 |
일단은 안내가 여러모로 부족하다는 의견이 있어서 최대한 친절하게 해보면 좋겠다는 것이 제 의견이구, 다른 분들도 의견 남겨주세요~~ |
멘트를 짧게해서 다음 진행사항 안내가 모호해 졌던거니까 멘트를 길게 쓰는 게 맞는 거 같네욥... 아예 멘트 자체에 말씀해주신대로 연락 방법을 명시하는건 좋은 거 같아요! 다만 진행하기 상태 변경에 대한 안내는 얼러트보다 모달 안쪽으로 넣는 건 어떠신가여? 저번 QA때 말이 나왔었던 얼러트가 사용자한테 불편함을 주는 거 같다는 의견에 동의해서요.. 모달 자체 크기를 좀 키워서라도 저는 성준님 말씀에 동의하구 다른분들도 말씀 주시면 반영할게요!! |
최상단 볼드처리 좋네요 그게 나을 것 같아여 |
일단 PR보류 하시면 안될까요? |
기획변경으로 인한 멘트 변경이 일어날 수 있는 상황이라고 생각합니다. |
말씀해주신 기획 변경이 어떤 경우 말씀해주시는 건가여? 해당 pr은 기존 사용자들이 불편하다고 말이 나왓어서 간단하게 멘트만 변경하는 걸로 일단 수정하는게 낫지 않나여?? 추후에 우려하신대로 기획 변경이 일어나도 크게 어려운 건 없는 거 같아서여 |
음 일단 그럼 머지 진행 후 우리 베타테스트중이라고 생각한다면 하시죠! |
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.
확인이 좀 늦었습니다 🥹 죄송해염 변경사항 확인했습니다! 수정 있을수도 있지만 큰 건들은 아닐 것 같으니 머지 편하게 하시라고 승인해두겠습니닷 (근데 저희 기획 변경에 대한 회의 ... 있나여?)
alert 불편하다는 의견 나와서 드리는 말씀인데 멘토가 멘티 요청 수락하는 부분에서도 모달로 슬랙 / 인트라 링크 띄우면서 연락해보라는 메세지 전하는 건 어떻게 생각하시나용? 멘티가 진행하기 눌렀을 때만 링크 뜨는 상황인데, 진행하기 누르는 건 왠지 … 맥도날드 쿠폰 사용하기 같은 느낌이라 진행 직전에 눌러야 할 것 같달까여 (진행 상태로 넘어간다는 개념은 저희만 공유하는 느낌 ... 제가 사용자(멘티)면 바로 진행해야 할 것 같은 느낌에 함부로 안 누를 것 같음)
그리고 멘트가 조금 길어지더라도 상태 변경 되었음과 연락 취해보라는 메세지 전달 같이 하는게 좋아보여여 내부 상태는 저희한테는 익숙하지만 사용자들에게는 그렇게 직관적으로 다가오지는 않을 수도 있다는 생각을 쪼끔 해봤습니다 ...
정리:
- 멘토 쪽에도 링크 연결되는 모달 띄우는 거 어떠시져
- 진행 상태(볼드 찬성) + 일정 잡아보세여 + 이동하시겠습니까 정도의 메세지 ... 좋아보임
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.
늦게 봐서 죄송합니다!!!
일단 코드는 다 확인 완료한 상황이고, 제 의견 몇 가지만 남겨드릴게요
- alert보다는 모달을 띄우는 것이 더 보기 좋을 것 같습니다. 뭔가 더 성의가 있다는 느낌을 받았어요
- 멘티가 진행하기 버튼을 눌렀을 때 멘토의 프로필을 확인하는 것처럼 멘토가 수락하기 버튼을 눌렀을 때 인트라 프로필을 확인하는 것 좋은것 같습니다. jinaji님 의견에 동의합니다.
- 상태가 바뀌었다고 글로만 쓰는 것보다 뭔가 다이어그램처럼 (요청) -> (수락하기) 이런식으로 상태가 바뀌었다는 것을 시각적으로 표현하는 것은 어떻게 생각하시나요? 작업이 그렇게 많이 늘어날 것 같지는 않아서 조심스럽게 제안 드립니다.
나머지는 문구만 바꾸면 되는 것 같은데 저는..
멘토가 멘티의 요청 수락시
(요청)->(수락)
수락되었습니다.
멘티와 멘토링 일정을 잡아주세요
멘티의 인트라 프로필 페이지로 이동하시겠습니까?
멘티가 멘토링 진행하기 버튼 클릭시
(수락)->(진행하기)
멘토와 멘토링 일정을 잡아주세요
멘토의 인트라 프로필 페이지로 이동하시겠습니까?
이렇게 생각하고 있습니다.
저번에 운영진 미팅하고 따로 변경해야할 것들에 대해서 이야기 해야하는데 그거 말씀하시는 것 같습니다.
수락하기에서도 모달로 처리하면 좋을 것 같아요.
요거 정확히 어떤건지 그림같은걸로 보여주시면 좋을 것 같습니다. @falconlee236 |
맨 위에 있는 그림 처럼을 말하는 것이였습니다!! @JuneParkCode |
"수락되었습니다" 라는 문장에서 이미 처리가 완료 됨을 알 수 있지 않을까요? |
상윤님이 말씀해주신 다이어그램같은 걸 추가하려면 제가 생각하기로는 멘토링 요청 -> 멘토 수락 -> 멘티가 진행하기 -> 멘토링 진행 -> 멘티 리뷰 -> 멘토 리뷰 -> 멘토링 종료 약간 쇼핑몰 주문하기 프로세스 다이어그램처럼 나타내는 컴포넌트가 있으면 유의미하게 작용할 거 같은데 |
우선 다른분들도 동일한 의견이신 멘토 alert는 변경하겠습니다 |
중앙 볼드처리보다는 그래도 그림으로 보는게 더 명확하다고 생각해서 제안 드렸습니다!! 불필요한 소요라면 이건 그대로 두고 alert먼저 해결해도 괜찮을거같습니다 |
수락된 상태에서 멘토링이 실제로 진행되기 전까지 진행하기 버튼을 누르지 않는 사용자가 있을 것 같다는 말이었습니다~! 현재는 그 부분에서만 소셜 링크를 통해 연락이 용이하게 돕는데, 요 부분까지 도달하지 않을 수도 있다는 말이었답니닷 그래서 그 앞 단계(수락하기) 에서도 모달 띄우는 것 말씀드린 거였어용 (수님 부탁해여~~ 🥹👍) |
캬~ 깔끔 |
멘 위 버튼 정말 좋은데요?? 역시 디자이너 출신인 것 같습니다! |
아니다, 이쁘네요 지금 다 삐뚤게 보이는 눈의 이슈가 있어서 ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ |
그럼 이대로 머지할까요?? |
@obvoso 머지하셔도 될 것 같긴한데 성준님이 팬딩걸어논거 같던데 이유가 뭘까요? |
위에 따봉해주셔서 그냥 해두 되는 거 같아욤 리뷰 감사합니다~ |
feat: 진행 상태 변경시 연락요청 메세지 추가 (#267) feat: 진행 상태 변경시 연락요청 메세지 추가 fix: 멘토의 수락 alert를 멘티의 진행 모달과 동일하게 수정 fix: modal close
멘토가 멘티의 요청 수락시
멘티가 멘토링 진행하기 버튼 클릭시