Skip to content

ref(admin): migrate forkCustomer off browserHistory#115915

Merged
evanpurkhiser merged 1 commit into
masterfrom
evanpurkhiser/ref-admin-migrate-forkcustomer-off-browserhistory
May 20, 2026
Merged

ref(admin): migrate forkCustomer off browserHistory#115915
evanpurkhiser merged 1 commit into
masterfrom
evanpurkhiser/ref-admin-migrate-forkcustomer-off-browserhistory

Conversation

@evanpurkhiser
Copy link
Copy Markdown
Member

Replace the browserHistory.push call in ForkCustomerAction with a navigate prop supplied by a function wrapper that calls useNavigate().

Drops browserHistory usages in static/ from 50 → 49.

Replace the `browserHistory.push` call in `ForkCustomerAction` with a navigate prop supplied by a function wrapper that calls `useNavigate()`.
@evanpurkhiser evanpurkhiser requested a review from a team as a code owner May 20, 2026 17:52
@evanpurkhiser evanpurkhiser enabled auto-merge (squash) May 20, 2026 17:52
@evanpurkhiser evanpurkhiser requested review from a team and removed request for a team May 20, 2026 17:52
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 20, 2026
@github-actions
Copy link
Copy Markdown
Contributor

📊 Type Coverage Diff

✅ No new type safety issues introduced. Coverage: 93.57%

@evanpurkhiser evanpurkhiser merged commit 8799c1f into master May 20, 2026
72 checks passed
@evanpurkhiser evanpurkhiser deleted the evanpurkhiser/ref-admin-migrate-forkcustomer-off-browserhistory branch May 20, 2026 17:59
JonasBa pushed a commit that referenced this pull request May 21, 2026
Replace the `browserHistory.push` call in `ForkCustomerAction` with a
navigate prop supplied by a function wrapper that calls `useNavigate()`.

Drops `browserHistory` usages in `static/` from 50 → 49.
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.

2 participants