Closed shgusgh12 closed 1 month ago
author.ts
의 getAuthorProfileImgSrc
에서 유저 아이콘 이미지가 없을 경우 img.onerror로 아래 코드와 같이 처리하고 있는 것 같은데 해당 코드 때문에 404가 나는 것이라고 말씀하신 것이 맞을까요 ?.?
img.onerror = () => {
img.src = `${GRAVATA_URL}/${md5(authorName)}}?d=identicon&f=y`;
};
author.ts
의getAuthorProfileImgSrc
에서 유저 아이콘 이미지가 없을 경우 img.onerror로 아래 코드와 같이 처리하고 있는 것 같은데 해당 코드 때문에 404가 나는 것이라고 말씀하신 것이 맞을까요 ?.?img.onerror = () => { img.src = `${GRAVATA_URL}/${md5(authorName)}}?d=identicon&f=y`; };
앗 제가 설명이 부족했네요
GRAVATA_URL
에서 가져오는 유저 아이콘일때 클릭시 github 유저 계정과 맞지 않아서 깃허브의 404 페이지로 이동하게 됩니다.
아 부분 불편하다고 생각하고 있었는데 넘 좋습니다..ㅎㅎ
했네요
GRAVATA_URL
에서 가져오는 유저 아이콘일때 클릭시 github 유저 계정과 맞지 않아서 깃허브의 404 페이지로 이동하게 됩니다.
아, 이제 기억이 났는데, ^^; 실제 git log 에 가져온 user정보와 github 계정정보가 매칭이 안되는 경우가 있어서 그렇습니다.
해당 부분에 대한 처리가 필요하겠네요. 클릭이 안되게 한다던지, 다른 메시지를 띄운다던지.. 등? 이것도 이슈 등록해주시면 감사하겠습니다!
유용한 기능이네요!! LGTM입니다!!
추가적으로, #668 의 리뷰에 있는 것처럼, 아래 사항들 고려해주시면 감사하겠습니다. (각각 이슈로 만들어주시면 좋을 것 같습니다!!)
- 현재는 외부 브라우저로 바로 링크가 열리는 것 같은데(맞죠?), vscode의 다른 panel로 열리거나, webview안에서 작은 frame으로 열리거나 하게 할 수도 있을 것 같아요. - 혹은 이런 외부 링크 열리는 방법들을 사용자 셋팅으로 결정되게 할 수도 있구요. - 그리고, 리뷰에 남긴 것처럼, github.com 외의 도메인을 지원하는 부분을 고려해야 할 수도 있을 것 같습니다.
넵 이슈에 추가하겠습니다!
했네요
GRAVATA_URL
에서 가져오는 유저 아이콘일때 클릭시 github 유저 계정과 맞지 않아서 깃허브의 404 페이지로 이동하게 됩니다.아, 이제 기억이 났는데, ^^; 실제 git log 에 가져온 user정보와 github 계정정보가 매칭이 안되는 경우가 있어서 그렇습니다.
해당 부분에 대한 처리가 필요하겠네요. 클릭이 안되게 한다던지, 다른 메시지를 띄운다던지.. 등? 이것도 이슈 등록해주시면 감사하겠습니다!
요것도 등록해놓겠습니다!
Related issue
571
Result
https://github.com/user-attachments/assets/4ffe3bcc-0e90-44a8-b0c6-2469c2665a05
Work list
Discussion
author.ts
의getAuthorProfileImgSrc
에서 유저 아이콘 이미지가 없는 경우GRAVATA_URL
에서 가져오는 경우 때문에 발생하는 것 같습니다.