Skip to content

netiquette - Don't suggest opening new issues if not possible#9028

Merged
fregante merged 2 commits intorefined-github:mainfrom
dnicolson:patch-4
Mar 11, 2026
Merged

netiquette - Don't suggest opening new issues if not possible#9028
fregante merged 2 commits intorefined-github:mainfrom
dnicolson:patch-4

Conversation

@dnicolson
Copy link
Contributor

This prevents the new issue link from being displayed if issues are disabled. rgh-netiquette is similarly affected, but the copy isn't as easily updated.

Test URLs

palera1n/loader#66

Screenshot

Before After
PixelSnap 2026-03-02 at 20 32 38@2x PixelSnap 2026-03-02 at 20 32 21@2x


export function getResolvedText(closingDate: Date): JSX.Element {
const ago = <strong>{twas(closingDate.getTime())}</strong>;
const canOpenNewIssue = elementExists('li:has([data-content="Issues"])');
Copy link
Member

Choose a reason for hiding this comment

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

@SunsetTechuila
Copy link
Member

In such cases, it might be better not to display the banner at all. What do you think @fregante?

@fregante
Copy link
Member

Yes 👆

@fregante fregante changed the title netiquette - Prevent new issue link if disabled netiquette - Don't suggest opening new issues if not possible Mar 11, 2026
@fregante fregante merged commit cbfd4da into refined-github:main Mar 11, 2026
11 checks passed
@SunsetTechuila
Copy link
Member

My feedback wasn't addressed

@fregante
Copy link
Member

Oh missed it

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

Development

Successfully merging this pull request may close these issues.

3 participants