`comments-time-machine-links` - Restore unnecessary notice suppression by SunsetTechuila · Pull Request #9409 · refined-github/refined-github · GitHub
Skip to content

comments-time-machine-links - Restore unnecessary notice suppression#9409

Merged
fregante merged 2 commits into
mainfrom
time-machine
May 10, 2026
Merged

comments-time-machine-links - Restore unnecessary notice suppression#9409
fregante merged 2 commits into
mainfrom
time-machine

Conversation

@SunsetTechuila

@SunsetTechuila SunsetTechuila commented May 9, 2026

Copy link
Copy Markdown
Contributor

@SunsetTechuila SunsetTechuila marked this pull request as ready for review May 10, 2026 13:37

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's the same now

'div[data-testid="latest-commit-details"] relative-time',
{waitForChildren: false},
);
if (lastCommitDate && date > lastCommitDate.getAttribute('datetime')!) {

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we may want to throw an error in case lastCommitDate in undefined

@SunsetTechuila SunsetTechuila changed the title comments-time-machine-links - Don't show notice when browsing older commits comments-time-machine-links - Don't show notice when not necessary May 10, 2026
@SunsetTechuila SunsetTechuila changed the title comments-time-machine-links - Don't show notice when not necessary comments-time-machine-links - Restore unnecessary notice suppression May 10, 2026
@fregante fregante merged commit eb3d2cd into main May 10, 2026
22 checks passed
@fregante

Copy link
Copy Markdown
Member

Not clear what the issue was though

@fregante fregante deleted the time-machine branch May 10, 2026 16:49
@SunsetTechuila

SunsetTechuila commented May 10, 2026

Copy link
Copy Markdown
Contributor Author
chrome_iNRGmoAecT

26.5.1:

image

Useless notice

@fregante

Copy link
Copy Markdown
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants