Skip to content

open-all-conversations - Fix multiple conversations detection#9506

Draft
SunsetTechuila wants to merge 8 commits into
mainfrom
open-all-conv
Draft

open-all-conversations - Fix multiple conversations detection#9506
SunsetTechuila wants to merge 8 commits into
mainfrom
open-all-conv

Conversation

@SunsetTechuila
Copy link
Copy Markdown
Member

@github-actions github-actions Bot added the bug label May 17, 2026
@SunsetTechuila

This comment was marked as resolved.

@SunsetTechuila SunsetTechuila marked this pull request as ready for review May 20, 2026 04:13
Comment thread source/features/open-all-conversations.tsx Outdated
Comment thread source/features/open-all-conversations.tsx Outdated
Comment thread source/features/open-all-conversations.tsx Outdated
Comment thread source/features/open-all-conversations.tsx Outdated
'[role="list"] > div:nth-child(2) > [class^="IssueRow"]',
const conversationCounterSelector = [
// PR list -- TODO: Drop after global and repo PR lists become exclusively React-based
'.table-list-header-toggle.states > a.selected',
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What if none is selected? Search "is:pr" only

@SunsetTechuila SunsetTechuila marked this pull request as draft May 20, 2026 05:51
@SunsetTechuila
Copy link
Copy Markdown
Member Author

This won't be ready for today's release

@SunsetTechuila
Copy link
Copy Markdown
Member Author

SunsetTechuila commented May 20, 2026

Unless you will pick up it and make sure it:

  • Works on beta global PR list
  • Works on repo and legacy global PR lists
  • Works on global and repo issue lists
  • Correctly handles lists without items
  • Works if the internet connection is slow

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.

open-all-conversations sometimes missing

2 participants