Skip to content

AVALON-33 CRM: Add Jetpack CRM Extensions to A4A's Dashboard #101997

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

Draft
wants to merge 17 commits into
base: trunk
Choose a base branch
from

Conversation

gogdzl
Copy link
Contributor

@gogdzl gogdzl commented Mar 27, 2025

Add Jetpack CRM Downloads Component to Multiple Calypso Sections

This PR adds the Jetpack CRM downloads to:

  1. Agencies for Agencies (A4A): In the licenses section

Integration Points

  • A4A: Integrated into the licenses section for agency users

Prerequisites

  • You need a Jetpack Complete purchase for one of your sites or, if you are an agency partner, have issued a Jetpack Complete license.

A4A Testing

  1. Log in as an agency user
  2. Navigate to the licenses section
  3. Find a license with Jetpack Complete
  4. Verify that:
    • The CRM section appears with the proper heading
    • The license key is displayed correctly
    • The extensions table shows available extensions
    • The extensions download OK

Error Handling Testing

  1. Test with an invalid or expired license key
  2. Verify that appropriate error messages are displayed

Screenshots

crm_a4a

Why are these changes being made?

These changes make it easier for people who purchase Jetpack Complete to download their extensions.

Pre-merge Checklist

  • Has the general commit checklist been followed? (PCYsg-hS-p2)
  • Have you written new tests for your changes?
  • Have you tested the feature in Simple (P9HQHe-k8-p2), Atomic (P9HQHe-jW-p2), and self-hosted Jetpack sites (PCYsg-g6b-p2)?
  • Have you checked for TypeScript, React or other console errors?
  • Have you used memoizing on expensive computations? More info in Memoizing with create-selector and Using memoizing selectors and Our Approach to Data
  • Have we added the "[Status] String Freeze" label as soon as any new strings were ready for translation (p4TIVU-5Jq-p2)?
    • For UI changes, have we tested the change in various languages (for example, ES, PT, FR, or DE)? The length of text and words vary significantly between languages.
  • For changes affecting Jetpack: Have we added the "[Status] Needs Privacy Updates" label if this pull request changes what data or activity we track or use (p4TIVU-aUh-p2)?

Copy link

github-actions bot commented Mar 27, 2025

Link to live branch is being generated...
Please wait a few minutes and refresh this page.

@gogdzl gogdzl self-assigned this Mar 27, 2025
@gogdzl gogdzl changed the base branch from trunk to add/crm-to-calypso-and-a4a March 27, 2025 18:40
@matticbot
Copy link
Contributor

This PR modifies the release build for the following Calypso Apps:

For info about this notification, see here: PCYsg-OT6-p2

  • help-center
  • notifications

To test WordPress.com changes, run install-plugin.sh $pluginSlug add/crm-to-a4a-only-second-try on your sandbox.

@mikestottuk mikestottuk requested a review from rcoll April 10, 2025 17:17
@matticbot matticbot added the [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. label Apr 10, 2025
@rcoll rcoll requested a review from heydemoura April 10, 2025 17:19
@heydemoura heydemoura force-pushed the add/crm-to-calypso-and-a4a branch 2 times, most recently from b2e5950 to e04559c Compare April 10, 2025 20:01
Base automatically changed from add/crm-to-calypso-and-a4a to trunk April 10, 2025 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants