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 console error in list-prs-for-file
#3946
Conversation
@@ -94,7 +94,7 @@ async function showTimemachineBar(): Promise<void | false> { | |||
const lastCommitDate = await elementReady([ | |||
'.repository-content .Box.Box--condensed relative-time', | |||
'[itemprop="dateModified"] relative-time' // "Repository refresh" layout | |||
].join(), {waitForChildren: false}); | |||
].join()); |
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.
I don’t think this is correct, we only use the attribute, not the children
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.
Also if you want to drop all pre-refresh code you can do so in a new PR, as long as no other changes are done in the same PR, because they’d be lost.
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.
I don’t think this is correct, we only use the attribute, not the children
Going to skip this for now as I don't have time to debug.
Also if you want to drop all pre-refresh code you can do so in a new PR, as long as no other changes are done in the same PR, because they’d be lost.
Sure
This reverts commit e909c45.
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.
🚀
Looks like I missed testing when there are no prs