Skip to content

update-pr-from-base-branch - Replace native button#9366

Merged
fregante merged 11 commits into
mainfrom
replace-native-update
May 7, 2026
Merged

update-pr-from-base-branch - Replace native button#9366
fregante merged 11 commits into
mainfrom
replace-native-update

Conversation

@SunsetTechuila
Copy link
Copy Markdown
Member

@SunsetTechuila SunsetTechuila commented May 7, 2026

Resolves #9365

Fixes #9229

Test URLs

Screenshot

pageDetect.isPRConversation,
],
exclude: [
pageDetect.isClosedConversation,
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Closed PR can be reopened

Copy link
Copy Markdown
Member

@fregante fregante left a comment

Choose a reason for hiding this comment

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

I like the change even though I'm hesitant to replace native buttons with API-based ones where a lot more can go wrong

Comment thread source/features/update-pr-from-base-branch.tsx
Comment thread source/features/update-pr-from-base-branch.tsx Outdated
Comment thread source/features/update-pr-from-base-branch.tsx
}

if (elementExists('.octicon-alert-fill', stateIcon)) {
// Button group won't exist if it wasn't previously added
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.

🙏

Comment thread source/features/update-pr-from-base-branch.tsx Outdated
Copy link
Copy Markdown
Member

@fregante fregante left a comment

Choose a reason for hiding this comment

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

Let's merge a few approved PRs

@fregante fregante merged commit 12e814a into main May 7, 2026
10 checks passed
@fregante fregante deleted the replace-native-update branch May 7, 2026 18:01
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.

update-pr-from-base-branch - Replace native button update-pr-from-base-branch duplicates native "resolve conflicts" dropdown

2 participants