[view] 사용자 정보 클릭시 해당 계정 페이지로 이동

[shgusgh12]

Related issue

#571

Result

2024-09-02.1.41.17.mov

Work list

• 유저 아이콘을 클릭시 유저의 깃허브 페이지로 이동합니다.

Discussion

• 특정 유저의 아이콘을 못찾는 경우에는 깃허브의 404 페이지로 이동하는 문제가 있습니다.
• author.ts의 getAuthorProfileImgSrc 에서 유저 아이콘 이미지가 없는 경우 GRAVATA_URL에서 가져오는 경우 때문에 발생하는 것 같습니다.
• Pr이 익숙하지 않아서 혹시 문제가 있다면 편하게 알려주시면 감사하겠습니다~!

[xxxjinn]

😮👍👍

[xxxjinn]

author.ts의 getAuthorProfileImgSrc에서 유저 아이콘 이미지가 없을 경우 img.onerror로 아래 코드와 같이 처리하고 있는 것 같은데 해당 코드 때문에 404가 나는 것이라고 말씀하신 것이 맞을까요 ?.?

img.onerror = () => {
  img.src = `${GRAVATA_URL}/${md5(authorName)}}?d=identicon&f=y`;
};

[shgusgh12]

author.ts의 getAuthorProfileImgSrc에서 유저 아이콘 이미지가 없을 경우 img.onerror로 아래 코드와 같이 처리하고 있는 것 같은데 해당 코드 때문에 404가 나는 것이라고 말씀하신 것이 맞을까요 ?.?

img.onerror = () => {
  img.src = `${GRAVATA_URL}/${md5(authorName)}}?d=identicon&f=y`;
};

앗 제가 설명이 부족했네요 GRAVATA_URL에서 가져오는 유저 아이콘일때 클릭시 github 유저 계정과 맞지 않아서 깃허브의 404 페이지로 이동하게 됩니다.

[ytaek]

유용한 기능이네요!! LGTM입니다!!

추가적으로, #668 의 리뷰에 있는 것처럼, 아래 사항들 고려해주시면 감사하겠습니다.
(각각 이슈로 만들어주시면 좋을 것 같습니다!!)

- 현재는 외부 브라우저로 바로 링크가 열리는 것 같은데(맞죠?), vscode의 다른 panel로 열리거나, webview안에서 작은 frame으로 열리거나 하게 할 수도 있을 것 같아요.
- 혹은 이런 외부 링크 열리는 방법들을 사용자 셋팅으로 결정되게 할 수도 있구요.
- 그리고, 리뷰에 남긴 것처럼, github.com 외의 도메인을 지원하는 부분을 고려해야 할 수도 있을 것 같습니다.

[ytaek]

github이외의 URL을 사용하는 (ex. github enterprise) 경우도 고려해주시면 좋겠습니다.
아래 건 참고하시면 되고,
#668 (comment)

해당 부분 이슈로 등록해주시면 감사하겠습니다!

[pcwadarong]

아 부분 불편하다고 생각하고 있었는데 넘 좋습니다..ㅎㅎ

[ytaek]

했네요 GRAVATA_URL에서 가져오는 유저 아이콘일때 클릭시 github 유저 계정과 맞지 않아서 깃허브의 404 페이지로 이동하게 됩니다.

아, 이제 기억이 났는데, ^^;
실제 git log 에 가져온 user정보와 github 계정정보가 매칭이 안되는 경우가 있어서 그렇습니다.

해당 부분에 대한 처리가 필요하겠네요. 클릭이 안되게 한다던지, 다른 메시지를 띄운다던지.. 등?
이것도 이슈 등록해주시면 감사하겠습니다!

[shgusgh12]

유용한 기능이네요!! LGTM입니다!!

추가적으로, #668 의 리뷰에 있는 것처럼, 아래 사항들 고려해주시면 감사하겠습니다. (각각 이슈로 만들어주시면 좋을 것 같습니다!!)

- 현재는 외부 브라우저로 바로 링크가 열리는 것 같은데(맞죠?), vscode의 다른 panel로 열리거나, webview안에서 작은 frame으로 열리거나 하게 할 수도 있을 것 같아요.
- 혹은 이런 외부 링크 열리는 방법들을 사용자 셋팅으로 결정되게 할 수도 있구요.
- 그리고, 리뷰에 남긴 것처럼, github.com 외의 도메인을 지원하는 부분을 고려해야 할 수도 있을 것 같습니다.

넵 이슈에 추가하겠습니다!

[shgusgh12]

했네요 GRAVATA_URL에서 가져오는 유저 아이콘일때 클릭시 github 유저 계정과 맞지 않아서 깃허브의 404 페이지로 이동하게 됩니다.

아, 이제 기억이 났는데, ^^; 실제 git log 에 가져온 user정보와 github 계정정보가 매칭이 안되는 경우가 있어서 그렇습니다.

해당 부분에 대한 처리가 필요하겠네요. 클릭이 안되게 한다던지, 다른 메시지를 띄운다던지.. 등? 이것도 이슈 등록해주시면 감사하겠습니다!

요것도 등록해놓겠습니다!