Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dev/core#3707 Fix wordpress issue of page not refreshing after import #23927

Merged
merged 1 commit into from
Jul 5, 2022

Conversation

eileenmcnaughton
Copy link
Contributor

@eileenmcnaughton eileenmcnaughton commented Jul 1, 2022

Overview

dev/core#3707 Fix wordpress issue of page not refreshing after import

https://lab.civicrm.org/dev/core/-/issues/3707

Before

After importing on wordpress the browser is redirect to what looks like the right url - but doesn't load up the right page unless you refresh it

After

It loads correctly,

Technical Details

The url was html-ised hence it looked right & loaded right on refresh

The other removed lines are a red herring - I originally thought they were the issue but they are actually now meaningless in the latest code. It used to be the case that the import eventually (& inconsistently) redirected back to the start page -but now it doesn't. (Apparent from confusing me when diagnosing this those lines also cause (more) issues with csv importer because they require a constant to be set that is not used in any meaningful way)

Comments

@civibot
Copy link

civibot bot commented Jul 1, 2022

(Standard links)

@rbaugh
Copy link

rbaugh commented Jul 5, 2022

@eileenmcnaughton This seems to have fixed the issue on my test of RC when importing. I am now presented the import results screen.

@demeritcowboy demeritcowboy merged commit c4a1233 into civicrm:5.51 Jul 5, 2022
@eileenmcnaughton eileenmcnaughton deleted the url branch July 5, 2022 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants