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

feat: 진행 상태 변경시 연락요청 메세지 추가 #267

Merged
merged 3 commits into from
Oct 31, 2023

Conversation

obvoso
Copy link
Contributor

@obvoso obvoso commented Oct 27, 2023

멘토가 멘티의 요청 수락시

스크린샷 2023-10-27 오후 8 01 13

멘티가 멘토링 진행하기 버튼 클릭시

스크린샷 2023-10-27 오후 7 57 19

@JuneParkCode
Copy link
Member

읽다보니까 조금 어색한 감이 있어서, 진행하기 쪽에서 멘트를 조금 바꿔보면 좋을 것 같아요.

  • 멘토링 진행하기 상태로 바뀜을 우선 알림
  • 멘토에게 연락을 취해달라고 안내
  • 아래 버튼을 누르면 이동할 수 있는 것을 안내

이렇게 나눠서 생각좀 해봐야할듯..!!
일단은 멘트만 바꾸면 되는거니 다른 분들 의견도 조금 들어봅시다~

@obvoso
Copy link
Contributor Author

obvoso commented Oct 27, 2023

씁 그렇네요... 글이 많아지는 걸 고려해서 문장의 순서를 바꾸는 건 어떨까요

멘토에게 연락을 취해주세요.
멘토의 인트라|슬랙 프로필로 이동하시겠습니까?

조금 어색할까요..?

@JuneParkCode
Copy link
Member

JuneParkCode commented Oct 27, 2023

조금 길어져도 괜찮을 것 같은데.. 이런건 어떨까요?

  • 우선 수락하기에서 연락을 취해보세요 보다
    "멘티에게 슬랙메시지를 보내 일정을 잡아주세요"
    라고 명확하게 안내해주면 좋을 것 같아요

  • 진행하기에서도 조금 더 명확하면 좋겠다는 생각이 들어요

진행하기 상태로 변경되었습니다. <- 이거 그냥 alert로라도 보여주면 이상할까요? 상태가 변경되었음을 알려야하지 않을까 생각이 드네요.

그리고 모달측은

"
아직 멘토와 연락이 닿지 않으셨나요?
멘토에게 슬랙메시지를 보내 일정을 잡아주세요.
인트라 | 슬랙 프로필로 이동하시겠습니까?
"

뭐 이런식으로 하면 좋을 것 같아요
그런데 멘트가 조금 어색한 것 같아서 고려좀 해봐야할듯..

@JuneParkCode
Copy link
Member

일단은 안내가 여러모로 부족하다는 의견이 있어서 최대한 친절하게 해보면 좋겠다는 것이 제 의견이구, 다른 분들도 의견 남겨주세요~~

@obvoso
Copy link
Contributor Author

obvoso commented Oct 27, 2023

멘트를 짧게해서 다음 진행사항 안내가 모호해 졌던거니까 멘트를 길게 쓰는 게 맞는 거 같네욥...

아예 멘트 자체에 말씀해주신대로 연락 방법을 명시하는건 좋은 거 같아요!

다만 진행하기 상태 변경에 대한 안내는 얼러트보다 모달 안쪽으로 넣는 건 어떠신가여?

저번 QA때 말이 나왔었던 얼러트가 사용자한테 불편함을 주는 거 같다는 의견에 동의해서요..

모달 자체 크기를 좀 키워서라도
"멘토링 진행 상태로 변경되었습니다." 라는 문구를 모달 최상단 중앙에 볼드체로 적고 말씀해주신 문구를 중앙부터 하단까지 위치하게 하는건 어떤가욥??

저는 성준님 말씀에 동의하구 다른분들도 말씀 주시면 반영할게요!!

@JuneParkCode
Copy link
Member

JuneParkCode commented Oct 27, 2023

최상단 볼드처리 좋네요 그게 나을 것 같아여

@JuneParkCode
Copy link
Member

@9utty @koreanddinghwan

@9utty
Copy link
Collaborator

9utty commented Oct 27, 2023

일단 PR보류 하시면 안될까요?

@9utty
Copy link
Collaborator

9utty commented Oct 27, 2023

기획변경으로 인한 멘트 변경이 일어날 수 있는 상황이라고 생각합니다.

@obvoso
Copy link
Contributor Author

obvoso commented Oct 27, 2023

말씀해주신 기획 변경이 어떤 경우 말씀해주시는 건가여?

해당 pr은 기존 사용자들이 불편하다고 말이 나왓어서 간단하게 멘트만 변경하는 걸로 일단 수정하는게 낫지 않나여??

추후에 우려하신대로 기획 변경이 일어나도 크게 어려운 건 없는 거 같아서여

@9utty
Copy link
Collaborator

9utty commented Oct 27, 2023

음 일단 그럼 머지 진행 후 우리 베타테스트중이라고 생각한다면 하시죠!

Copy link
Contributor

@jinaji jinaji left a comment

Choose a reason for hiding this comment

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

확인이 좀 늦었습니다 🥹 죄송해염 변경사항 확인했습니다! 수정 있을수도 있지만 큰 건들은 아닐 것 같으니 머지 편하게 하시라고 승인해두겠습니닷 (근데 저희 기획 변경에 대한 회의 ... 있나여?)

alert 불편하다는 의견 나와서 드리는 말씀인데 멘토가 멘티 요청 수락하는 부분에서도 모달로 슬랙 / 인트라 링크 띄우면서 연락해보라는 메세지 전하는 건 어떻게 생각하시나용? 멘티가 진행하기 눌렀을 때만 링크 뜨는 상황인데, 진행하기 누르는 건 왠지 … 맥도날드 쿠폰 사용하기 같은 느낌이라 진행 직전에 눌러야 할 것 같달까여 (진행 상태로 넘어간다는 개념은 저희만 공유하는 느낌 ... 제가 사용자(멘티)면 바로 진행해야 할 것 같은 느낌에 함부로 안 누를 것 같음)
그리고 멘트가 조금 길어지더라도 상태 변경 되었음과 연락 취해보라는 메세지 전달 같이 하는게 좋아보여여 내부 상태는 저희한테는 익숙하지만 사용자들에게는 그렇게 직관적으로 다가오지는 않을 수도 있다는 생각을 쪼끔 해봤습니다 ...

정리:

  • 멘토 쪽에도 링크 연결되는 모달 띄우는 거 어떠시져
  • 진행 상태(볼드 찬성) + 일정 잡아보세여 + 이동하시겠습니까 정도의 메세지 ... 좋아보임

Copy link
Member

@falconlee236 falconlee236 left a comment

Choose a reason for hiding this comment

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

늦게 봐서 죄송합니다!!!
일단 코드는 다 확인 완료한 상황이고, 제 의견 몇 가지만 남겨드릴게요

  1. alert보다는 모달을 띄우는 것이 더 보기 좋을 것 같습니다. 뭔가 더 성의가 있다는 느낌을 받았어요
  2. 멘티가 진행하기 버튼을 눌렀을 때 멘토의 프로필을 확인하는 것처럼 멘토가 수락하기 버튼을 눌렀을 때 인트라 프로필을 확인하는 것 좋은것 같습니다. jinaji님 의견에 동의합니다.
  3. 상태가 바뀌었다고 글로만 쓰는 것보다 뭔가 다이어그램처럼 (요청) -> (수락하기) 이런식으로 상태가 바뀌었다는 것을 시각적으로 표현하는 것은 어떻게 생각하시나요? 작업이 그렇게 많이 늘어날 것 같지는 않아서 조심스럽게 제안 드립니다.

나머지는 문구만 바꾸면 되는 것 같은데 저는..
멘토가 멘티의 요청 수락시
(요청)->(수락)
수락되었습니다.
멘티와 멘토링 일정을 잡아주세요
멘티의 인트라 프로필 페이지로 이동하시겠습니까?

멘티가 멘토링 진행하기 버튼 클릭시
(수락)->(진행하기)
멘토와 멘토링 일정을 잡아주세요
멘토의 인트라 프로필 페이지로 이동하시겠습니까?

이렇게 생각하고 있습니다.

@JuneParkCode
Copy link
Member

근데 저희 기획 변경에 대한 회의 ... 있나여?

저번에 운영진 미팅하고 따로 변경해야할 것들에 대해서 이야기 해야하는데 그거 말씀하시는 것 같습니다.

alert 불편하다는 의견 나와서 드리는 말씀인데 멘토가 멘티 요청 수락하는 부분에서도 모달로 슬랙 / 인트라 링크 띄우면서 연락해보라는 메세지 전하는 건 어떻게 생각하시나용? 멘티가 진행하기 눌렀을 때만 링크 뜨는 상황인데, 진행하기 누르는 건 왠지 … 맥도날드 쿠폰 사용하기 같은 느낌이라 진행 직전에 눌러야 할 것 같달까여 (진행 상태로 넘어간다는 개념은 저희만 공유하는 느낌 ... 제가 사용자(멘티)면 바로 진행해야 할 것 같은 느낌에 함부로 안 누를 것 같음)

수락하기에서도 모달로 처리하면 좋을 것 같아요.
진행하기 상태에 대해서는 조금 어떤 말씀인지 잘 와닿지가 않는데 다시 이야기해주실 수 있을까요? @jinaji

상태가 바뀌었다고 글로만 쓰는 것보다 뭔가 다이어그램처럼 (요청) -> (수락하기) 이런식으로 상태가 바뀌었다는 것을 시각적으로 표현하는 것은 어떻게 생각하시나요

요거 정확히 어떤건지 그림같은걸로 보여주시면 좋을 것 같습니다. @falconlee236

@falconlee236
Copy link
Member

falconlee236 commented Oct 31, 2023

요거 정확히 어떤건지 그림같은걸로 보여주시면 좋을 것 같습니다

image

맨 위에 있는 그림 처럼을 말하는 것이였습니다!! @JuneParkCode

@JuneParkCode
Copy link
Member

JuneParkCode commented Oct 31, 2023

"수락되었습니다" 라는 문장에서 이미 처리가 완료 됨을 알 수 있지 않을까요?
송민님이 말씀해주신 것처럼 중앙 볼드처리를 해두면 명확하게 되지 않을까... 싶어서요

@obvoso
Copy link
Contributor Author

obvoso commented Oct 31, 2023

상윤님이 말씀해주신 다이어그램같은 걸 추가하려면 제가 생각하기로는
전체 프로세스를 나타내는 다이어그램이 필요할 거 같아요

멘토링 요청 -> 멘토 수락 -> 멘티가 진행하기 -> 멘토링 진행 -> 멘티 리뷰 -> 멘토 리뷰 -> 멘토링 종료

약간 쇼핑몰 주문하기 프로세스 다이어그램처럼 나타내는 컴포넌트가 있으면 유의미하게 작용할 거 같은데
현 pr처럼 멘토수락 -> 멘티 진행하기 이 두 부분에서만 나타내기에는 조금 아쉬운 거 같아요...

@obvoso
Copy link
Contributor Author

obvoso commented Oct 31, 2023

우선 다른분들도 동일한 의견이신 멘토 alert는 변경하겠습니다
멘토의 요청 수락에서 alert대신 멘티측 진행하기 모달처럼 변경하겠습니다~

@falconlee236
Copy link
Member

"수락되었습니다" 라는 문장에서 이미 처리가 완료 됨을 알 수 있지 않을까요?
송민님이 말씀해주신 것처럼 중앙 볼드처리를 해두면 명확하게 되지 않을까... 싶어서요

중앙 볼드처리보다는 그래도 그림으로 보는게 더 명확하다고 생각해서 제안 드렸습니다!! 불필요한 소요라면 이건 그대로 두고 alert먼저 해결해도 괜찮을거같습니다

@jinaji
Copy link
Contributor

jinaji commented Oct 31, 2023

진행하기 상태에 대해서는 조금 어떤 말씀인지 잘 와닿지가 않는데 다시 이야기해주실 수 있을까요?

수락된 상태에서 멘토링이 실제로 진행되기 전까지 진행하기 버튼을 누르지 않는 사용자가 있을 것 같다는 말이었습니다~! 현재는 그 부분에서만 소셜 링크를 통해 연락이 용이하게 돕는데, 요 부분까지 도달하지 않을 수도 있다는 말이었답니닷 그래서 그 앞 단계(수락하기) 에서도 모달 띄우는 것 말씀드린 거였어용 (수님 부탁해여~~ 🥹👍)

@obvoso
Copy link
Contributor Author

obvoso commented Oct 31, 2023

멘토가 멘티의 요청 수락시

스크린샷 2023-10-31 오후 6 27 54

멘티가 멘토링 진행하기 버튼 클릭시

스크린샷 2023-10-31 오후 6 29 50

모달 변경 push했습니다. 확인 부탁드려여!

  • 진행(변경) 상태 명시
  • 수락시 멘티의 인트라 프로필로 이동
  • 연락해달라는 멘트 추가

@JuneParkCode @falconlee236 @jinaji

@JuneParkCode
Copy link
Member

캬~ 깔끔

@falconlee236
Copy link
Member

멘 위 버튼 정말 좋은데요?? 역시 디자이너 출신인 것 같습니다!

@9utty
Copy link
Collaborator

9utty commented Oct 31, 2023

아니다, 이쁘네요 지금 다 삐뚤게 보이는 눈의 이슈가 있어서 ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ

@obvoso
Copy link
Contributor Author

obvoso commented Oct 31, 2023

그럼 이대로 머지할까요??

@9utty
Copy link
Collaborator

9utty commented Oct 31, 2023

@obvoso 머지하셔도 될 것 같긴한데 성준님이 팬딩걸어논거 같던데 이유가 뭘까요?

@obvoso
Copy link
Contributor Author

obvoso commented Oct 31, 2023

@obvoso 머지하셔도 될 것 같긴한데 성준님이 팬딩걸어논거 같던데 이유가 뭘까요?

위에 따봉해주셔서 그냥 해두 되는 거 같아욤 리뷰 감사합니다~

@obvoso obvoso self-assigned this Oct 31, 2023
@obvoso obvoso merged commit f19b195 into manito42:dev Oct 31, 2023
1 check passed
@JuneParkCode JuneParkCode mentioned this pull request Nov 1, 2023
JuneParkCode added a commit that referenced this pull request Nov 1, 2023
feat: 진행 상태 변경시 연락요청 메세지 추가 (#267)

feat: 진행 상태 변경시 연락요청 메세지 추가

fix: 멘토의 수락 alert를 멘티의 진행 모달과 동일하게 수정

fix: modal close
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.

5 participants