Skip to content

update-pr-from-base-branch - Improve reliability#8831

Merged
fregante merged 1 commit intorefined-github:mainfrom
biplavbarua:fix/pr-update-button-selector
Jan 3, 2026
Merged

update-pr-from-base-branch - Improve reliability#8831
fregante merged 1 commit intorefined-github:mainfrom
biplavbarua:fix/pr-update-button-selector

Conversation

@biplavbarua
Copy link
Contributor

Fixes #8828.
The existing selector relied on specific text content (caption) within the merge box. This updates it to target the stable container class (MergeBoxSectionHeader-module__wrapper), ensuring the button appears even if the text varies or is missing.

@github-actions github-actions bot added the bug label Dec 22, 2025
@github-actions github-actions bot changed the title fix: use robust selector for update-pr-from-base-branch Use robust selector for update-pr-from-base-branch Dec 22, 2025
@github-actions github-actions bot changed the title Use robust selector for update-pr-from-base-branch Use robust selector for update-pr-from-base-branch Dec 22, 2025
@biplavbarua biplavbarua force-pushed the fix/pr-update-button-selector branch from 85e23a0 to 2852786 Compare December 22, 2025 13:17
@biplavbarua biplavbarua force-pushed the fix/pr-update-button-selector branch from 2852786 to 1d6aaac Compare December 22, 2025 13:26
@fregante fregante changed the title Use robust selector for update-pr-from-base-branch update-pr-from-base-branch - Improve reliability Jan 3, 2026
observe([
'.mergeability-details > *:last-child', // Old view - TODO: Drop after June 2025
prMergeabilityBoxCaption,
prMergeabilityBoxHeader,
Copy link
Member

Choose a reason for hiding this comment

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

Can't repro the original issue and sadly the element that this selector selects isn't even used by addButton.

Merging it but ideally we'd drop the legacy code soon and use observe correctly

@fregante fregante merged commit e3cb2e9 into refined-github:main Jan 3, 2026
14 checks passed
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.

update-pr-from-base-branch often not shown, with no clues as to why not

2 participants