My Jetpack: Requires/Push to connection only if needed

[renatoagds]

Closes #27608
Closes #27612
Closes #27611
Closes #27695

Changes proposed in this Pull Request:

• Push user to connection page only when activating and only if needed.
• Update ProductDetailCard to use useProductCheckoutWorkflow.
• Update get_products_from_wpcom to use endpoint based on user connection.

Other information:

• Have you written new tests for your changes, if applicable?
• Have you checked the E2E test CI results, and verified that your changes do not break them?

Jetpack product discussion

N/A

Does this pull request change what data or activity we track or use?

No.

Testing instructions:

Fully Disconnected with Jetpack plugin

⚠️ Make sure that you're fully or user disconnected.

• Navigate to My Jetpack.
• You should be able to:

• • Add (install) a new product that doesn't require purchase, and be redirect to it (CRM, Akismet for ex.)
• • Purchase a new product, being requested to connect first (Backup, Search for ex.).
• • Activate a module, being moved to connect screen first.

Fully/User Disconnected with Standalone plugin

⚠️ Make sure that you're fully or user disconnected.

• Navigate to My Jetpack.
• You should be able to:

• • Add (install) a new product that doesn't require purchase, and be redirect to it (CRM, Akismet for ex.)
• • Purchase a new product, being requested to connect first (Backup, Search for ex.).

[github-actions]

Are you an Automattician? You can now test your Pull Request on WordPress.com. On your sandbox, run bin/jetpack-downloader test jetpack update/my-jetpack-activate-while-disconnected to get started. More details: p9dueE-5Nn-p2

[github-actions]

Thank you for your PR!

When contributing to Jetpack, we have a few suggestions that can help us test and review your patch:

• ✅ Include a description of your PR changes.
  
• ✅ All commits were linted before commit.
  
• ✅ Add a "[Status]" label (In Progress, Needs Team Review, ...).
  
• ✅ Add testing instructions.
  
• ✅ Specify whether this PR includes any changes to data or privacy.
  
• ✅ Add changelog entries to affected projects
  

This comment will be updated as you work on your PR and make changes. If you think that some of those checks are not needed for your PR, please explain why you think so. Thanks for cooperation 🤖

---

The e2e test report can be found here. Please note that it can take a few minutes after the e2e tests checks are complete for the report to be available.

---

Once your PR is ready for review, check one last time that all required checks (other than "Required review") appearing at the bottom of this PR are passing or skipped.
Then, add the "[Status] Needs Team review" label and ask someone from your team review the code.
Once you’ve done so, switch to the "[Status] Needs Review" label; someone from Jetpack Crew will then review this PR and merge it to be included in the next Jetpack release.

[nunyvega]

I'm getting this error in the console when opening My Jetpack

[nunyvega]

Not sure if it's related to the changes that we made in this project, but the Add Security button does not work on this screen:

[nunyvega]

I see that VideoPress asks for a connection instead of showing the plan's price and a "Start for free" option. We can think about adding something like what Search has in a follow-up PR:

[nunyvega]

All the others products and cases that I tested worked as expected. Reviewing the code now

[nunyvega]

Code changes LGTM

[oskosk]

LGTM!

[sdixon194]

Cherry picked and released new versions of My Jetpack 2.5.0.1 to all current release branches - Jetpack, Social, and Backup.