-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
linkify-code
- Fix issue links on global search pages
#7005
Conversation
linkifyIssues(currentRepo, element); | ||
} | ||
const currentRepo = pageDetect.isGlobalSearchResults() | ||
? getRepo(wrapper.parentElement!.querySelector('.blob-num a')!.href) |
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.
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.
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.
Oh neat, I thought it was targeting the file header
linkify-code
- Fix on search pageslinkify-code
- Fix issue links on global search pages
Fixes #6996
Retrieves the repository using the line url when getRepo doesn't work
/notion-enhancer/notion-repackaged/blob/4eefa99dc1bcde18268d70bb2b7f050786229feb/scripts/extract-src.sh#L50
↑ pull repo from this
Test URLs
https://github.com/search?q=repo%3Anotion-enhancer%2Fnotion-repackaged%20sqlite&type=code
https://github.com/search?q=org%3Arefined-github+%2F%23%5Cd%2B%2F&type=code
https://github.com/search?q=%2F%23%5Cd%7B4%2C%7D%2F+language%3Atypescript&type=code
Screenshot