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

Remove Jetpack + Connected Services by WooCommerce #4

Closed
nomadicjosh opened this issue Apr 3, 2019 · 9 comments · Fixed by #30, #105 or #121
Closed

Remove Jetpack + Connected Services by WooCommerce #4

nomadicjosh opened this issue Apr 3, 2019 · 9 comments · Fixed by #30, #105 or #121
Assignees
Labels
help wanted Extra attention is needed question Further information is requested

Comments

@nomadicjosh
Copy link
Contributor

I am sure Jetpack is going to get a Gutenberg update sometime in the future if it hasn't already. But one of the great features of the Jetpack and WooCommerce combination is the ability to print shipping labels.

Should Jetpack and WooCommerce connection be left for now or replaced? If replaced, what alternatives are there for shipping?

@nomadicjosh nomadicjosh added the help wanted Extra attention is needed label Apr 3, 2019
@nomadicjosh nomadicjosh modified the milestone: Version 1.0.0 Apr 3, 2019
@nylen
Copy link
Contributor

nylen commented Apr 3, 2019

At the very least you'd need to ask WooCommerce support for permission to continue using this, otherwise, the Jetpack <-> WooCommerce connection and any features that depend on it should probably be removed until we can build our own replacement for them.

@nylen nylen mentioned this issue Apr 3, 2019
@zulfgani
Copy link
Contributor

zulfgani commented Apr 3, 2019

I'd advocate for the removal of the connection in favor of building our own.

There's also the connection to the Woo Services that will need removing or at least consideration on it's continued use. (This might cross with the JetPack connection as I've not looked into the workings of it).

@nomadicjosh
Copy link
Contributor Author

@zulfgani I agree, especially in regards to the recent Jetpack news. I think it is best to remove all Woocommerce.com connections as well as Jetpack connections.

@bahiirwa bahiirwa added this to the Version 1.0.0 milestone Jun 24, 2019
@bahiirwa bahiirwa self-assigned this Sep 28, 2019
@bahiirwa bahiirwa changed the title Jetpack Remove Jetpack + Connected Services by WooCommerce Sep 28, 2019
@bahiirwa
Copy link
Collaborator

bahiirwa commented Sep 28, 2019

More interconnceted services that need removal are here

  • WooCommerce Services
  • Stripe Payment
  • Automated taxes
  • PayPal

Some of these most likely will be plugged in a individual plugins from the conversation in #8

@nylen
Copy link
Contributor

nylen commented Oct 2, 2019

Stripe and PayPal are widely used. If they are already included (looks like PayPal is, but Stripe is not) then why remove them?

@ghost
Copy link

ghost commented Oct 4, 2019

I was also wondering why you are considering removing the Paypal Standard option that comes with the initial install. I don't think this is linked to Jetpack at all and it's quite handy for a very simple shop setup.

@bahiirwa bahiirwa added question Further information is requested Has PR Issue has a PR. and removed Needs Feedback labels Oct 9, 2019
@bahiirwa bahiirwa removed the Has PR Issue has a PR. label Oct 28, 2019
@bahiirwa bahiirwa reopened this Oct 29, 2019
@bahiirwa
Copy link
Collaborator

bahiirwa commented Oct 29, 2019

I was also wondering why you are considering removing the Paypal Standard

This stays in the plugin and can be activated in the payments tab under settings.

I don't think this is linked to Jetpack at all

PayPal is not but PayPal Pro is tied to woo services if installed in the wizard. Plugin installs and themes in the wizard have been removed as of #30

@dougaitken
Copy link
Contributor

but PayPal Pro is tied to woo services if installed in the wizard

PayPal Pro has no connection to "woo services".

As a small note: No payment gateway installed via the WooCommerce Onboarding process has a requirement for Jetpack.

@bahiirwa bahiirwa reopened this Nov 9, 2019
@bahiirwa
Copy link
Collaborator

Closed in #121

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