-
Notifications
You must be signed in to change notification settings - Fork 102
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
fix: repo homepage latest commit hovercard openrank display #846
Conversation
try { | ||
await elementReadyWithTimeout('[data-testid=github-avatar]', { stopOnDomReady: false }, 1500); | ||
} catch (error) { | ||
console.error(error); |
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.
Is it necessary to output error messages here?
Why does the function sometimes load twice. Does @wxharry know the reason? |
Do you know how to replicate the issue? I thought it was handled by the previous PR |
![]() ![]() This feature sometimes occurs when entering a repository, and it is loaded twice. |
Sorry, I don't what happened, I merged this branch. I just reverted you changes back, but this PR is closed, could you make an other PR and mention this one? Sorry for the inconvenience and thank you so much! |
OK, I'll resubmit the PR later. |
Brief Information
This pull request is in the type of (more info about types):
Related issues #842
Details
Checklist
Others
Thanks to Yantong's help, OpenRank can now be displayed normally in most scenarios.