Skip to content

ref(onboarding): migrate createSampleEventButton off browserHistory#115921

Closed
evanpurkhiser wants to merge 1 commit into
masterfrom
evanpurkhiser/ref-onboarding-migrate-createsampleeventbutton-off-browserhistory
Closed

ref(onboarding): migrate createSampleEventButton off browserHistory#115921
evanpurkhiser wants to merge 1 commit into
masterfrom
evanpurkhiser/ref-onboarding-migrate-createsampleeventbutton-off-browserhistory

Conversation

@evanpurkhiser
Copy link
Copy Markdown
Member

Replace the browserHistory.push call in CreateSampleEventButton with a navigate prop supplied by a function wrapper that calls useNavigate(). Split the props type so the public ButtonProps union stays intact for callers.

Replace the `browserHistory.push` call in `CreateSampleEventButton` with a navigate prop supplied by a function wrapper that calls `useNavigate()`. Split the props type so the public `ButtonProps` union stays intact for callers.
@evanpurkhiser evanpurkhiser requested a review from a team as a code owner May 20, 2026 18:09
@evanpurkhiser evanpurkhiser enabled auto-merge (squash) May 20, 2026 18:09
@evanpurkhiser evanpurkhiser requested review from a team and removed request for a team May 20, 2026 18:09
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 20, 2026
@evanpurkhiser
Copy link
Copy Markdown
Member Author

Superseded by #115830 which converted this file to a functional component using useNavigate directly.

auto-merge was automatically disabled May 20, 2026 19:27

Pull request was closed

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

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant