[wp-trac] [WordPress Trac] #61330: Plugin Dependencies: Refresh plugin-install.php instead of redirecting on activation
WordPress Trac
noreply at wordpress.org
Mon Jun 3 20:25:29 UTC 2024
#61330: Plugin Dependencies: Refresh plugin-install.php instead of redirecting on
activation
------------------------------------------------+--------------------------
Reporter: jorbin | Owner: (none)
Type: enhancement | Status: new
Priority: normal | Milestone: 6.7
Component: Upgrade/Install | Version:
Severity: normal | Resolution:
Keywords: 2nd-opinion has-patch dev-feedback | Focuses:
| administration
------------------------------------------------+--------------------------
Changes (by costdev):
* keywords: 2nd-opinion has-patch => 2nd-opinion has-patch dev-feedback
Comment:
I've pushed up [https://github.com/WordPress/wordpress-develop/pull/6717
PR 6717].
Known issue:
- If a plugin redirects on page load, such as to onboarding, the browser's
`back` button will navigate to the page ''before'' the one that was used
to activate the plugin.
- Pushing a new entry to the browser's history can help resolve this.
However, a new entry should only be pushed after the page reload, and only
if a redirect is performed. It was proving difficult for me to identify
how this could be advanced, so I pushed the reload up as PR for further
investigation.
@jorbin @hellofromTonya @afragen It's my current view that finding a
workable approach for this may take more time. At the moment, I'm thinking
this isn't achievable for 6.6.
However, we ''could'' commit the PR and treat the known issue as a bug to
fix during 6.6 Beta. This would give us more time without punting the
ticket. If it isn't fixed as we approach 6.6 RC, we could then just revert
it. I'm just mentioning this as a possibility, even though I'm leaning
towards omitting from 6.6 at this time.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/61330#comment:4>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list