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

VIP Go setup #500

Closed
jonny-bull opened this issue Dec 20, 2019 · 8 comments · Fixed by #606
Closed

VIP Go setup #500

jonny-bull opened this issue Dec 20, 2019 · 8 comments · Fixed by #606
Labels
help wanted type:bug Something isn't working.
Milestone

Comments

@jonny-bull
Copy link

jonny-bull commented Dec 20, 2019

Describe your question

I've got a WP multisite set up locally with Distributor that works as expected.

When deploying to a VIP Go environment, the sites in my network aren't detected.

Is there any extra setup required to get networked sites to appear as options for pulling and pushing on VIP Go?

@jonny-bull jonny-bull added the type:question Further information is requested. label Dec 20, 2019
@jonny-bull jonny-bull changed the title VIP setup VIP Go setup Dec 20, 2019
@jeffpaul
Copy link
Member

@jonny-bull welcome to Distributor and great question! Unfortunately I don't have an immediate answer for you, though you might try the WPCOM OAuth2 connections to see if that works while I dig a bit to get you a better answer.

Note that 10up is closed for the holidays starting tomorrow through Wednesday, January 1st so it'll likely be early January at best before I'm able to get back to you with an answer. Apologies for the delay, but hopefully you'll have a safe and happy holiday in the meantime... cheers!

@jeffpaul jeffpaul self-assigned this Dec 20, 2019
@jeffpaul
Copy link
Member

jeffpaul commented Jan 6, 2020

@jonny-bull were you able to try using the WPCOM OAuth2 connections to see if that resolved things for you?

@jeffpaul jeffpaul added the needs:feedback This requires reporter feedback to better understand the request. label Jan 6, 2020
@jeffpaul
Copy link
Member

@jonny-bull were you able to try using the WPCOM OAuth2 connections to see if that resolved things for you?

@WPprodigy
Copy link

WPprodigy commented Jan 30, 2020

Hey @jeffpaul,

I believe this is what might have been tripping them up:

distributor/distributor.php

Lines 110 to 112 in 2e8e077

if ( \Distributor\Utils\is_vip_com() ) {
add_filter( 'dt_network_site_connection_enabled', '__return_false', 9 );
}
Distributor\Utils\is_vip_com() is still true for VIP Go sites, but seems only necessary for WPcom VIP Sites.

Could perhaps also check if the VIP_GO_ENV constant is set and is not false: https://github.com/Automattic/vip-go-mu-plugins/blob/abcc27185d0b163f66153575cdd4cd41148deb26/000-vip-init.php#L66.

Alternatively, I think we've got everybody migrated to VIP Go that is using this plugin, so I could double check on that and this logic could potentially just be removed entirely?

@jeffpaul
Copy link
Member

@WPprodigy thanks for the notes, much appreciated! I'd rather play it cautious and avoid any mistake in a VIP site getting caught here, so the VIP_GO_ENV check you noted make be the best route.

@jeffpaul jeffpaul added type:bug Something isn't working. help wanted and removed needs:feedback This requires reporter feedback to better understand the request. type:question Further information is requested. labels Jan 31, 2020
@jeffpaul jeffpaul added this to the 2.0.0 milestone Jan 31, 2020
@jeffpaul
Copy link
Member

jeffpaul commented Jun 8, 2020

@WPprodigy any chance you've been able to check to see if everybody using Distributor has been migrated to VIP Go?

@jeffpaul jeffpaul modified the milestones: 1.6.0, 2.0.0 Jun 8, 2020
@WPprodigy
Copy link

Confirmed! All on VIP Go.

@jeffpaul jeffpaul assigned dinhtungdu and unassigned jeffpaul Jun 8, 2020
@jeffpaul jeffpaul modified the milestones: 2.0.0, 1.6.0 Jun 8, 2020
@helen helen modified the milestones: 2.0.0, 1.6.0 Jun 16, 2020
@jeffpaul
Copy link
Member

jeffpaul commented Jul 2, 2020

In discussing the best approach here, the decision was to remove the logic noted in the #500 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted type:bug Something isn't working.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants