Skip to content

fix: watch/fork/star popups adapt for new DOM, and add github restriction tips#44

Merged
rrbe merged 5 commits into
mainfrom
fix/watch-fork-star-popup-dom
Jul 23, 2026
Merged

fix: watch/fork/star popups adapt for new DOM, and add github restriction tips#44
rrbe merged 5 commits into
mainfrom
fix/watch-fork-star-popup-dom

Conversation

@rrbe

@rrbe rrbe commented Jul 23, 2026 •
edited
Loading

Copy link
Copy Markdown
Owner

Summary

  • keep support for the legacy pagehead counters
  • target the redesigned Watch, Fork, and Star counters through their semantic data-testid and data-component attributes
  • check authenticated-token repository permissions before loading Stargazers or Watchers
  • skip restricted list endpoints for non-collaborators and link to the GitHub access-restriction notice
  • cover the live GitHub React action markup and restricted-state behavior with regression tests

Verification

  • confirmed the selectors against a signed-in GitHub page with Chrome DevTools MCP
  • pnpm test
  • pnpm typecheck
  • pnpm build

@rrbe rrbe changed the title Fix watch, fork, and star popups for new repository actions fix: watch, fork, and star popups for new repository actions Jul 23, 2026
@rrbe rrbe changed the title fix: watch, fork, and star popups for new repository actions fix: watch/fork/star popups adapt for new DOM, and add github restriction tips Jul 23, 2026
@rrbe
rrbe merged commit fb4d225 into main Jul 23, 2026
1 check passed
@rrbe
rrbe deleted the fix/watch-fork-star-popup-dom branch July 23, 2026 04:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@rrbe