rriemann
(Robert)
1
Hello,
for the link https://github.com/beta-europe/www.beta-europe.org/commits/master, I expected to have a onebox different from a precise commit. This is a bit misleading.
Onebox for the link above:
(edit by @david: screenshot of the old result, for future reference)
4개의 좋아요
sam
(Sam Saffron)
2
Agree, we can do better here pr-welcome
3개의 좋아요
maja
(Maja)
3
Links leading to issues and pull requests overview both render whitelistedgeneric onebox:
https://github.com/rails/rails/pulls
Should the same happen when adding a link to commit history as well?
1개의 좋아요
david
(David Taylor)
5
Maja가 제안한 대로, 여러 커밋에 대한 링크는 이제 화이트리스트에 등록된 일반 onebox로 렌더링됩니다.
불행히도 GitHub가 이러한 URL에 제공하는 오픈그래프 데이터는 일반적입니다. 리포지토리의 이름/설명만 표시됩니다. 하지만 여전히 첫 번째 커밋을 임의로 렌더링하는 것보다 혼란이 적다고 생각합니다.
이러한 “다중 커밋” 링크를 위한 완전히 새로운 onebox는 여전히 #pr-welcome입니다. 시도해 보시려는 분이 있다면 환영합니다.
5개의 좋아요