Skip to content

Allow GravityFormsCLI to be Installed by Packagist#49

Merged
ibd21 merged 6 commits intomasterfrom
fix-4938-packagist-compatibility
Mar 17, 2026
Merged

Allow GravityFormsCLI to be Installed by Packagist#49
ibd21 merged 6 commits intomasterfrom
fix-4938-packagist-compatibility

Conversation

@ibd21
Copy link
Contributor

@ibd21 ibd21 commented Mar 11, 2026

Description

This PR works to resolve issue #4938, through the support of PRs #47 & #48. With this work, we are allowing the Gravity Forms CLI package to be accessible and installable as a standalone WP-CLI package via WPackagist.

Next Steps needed: Once merged, we need to submit the package to Packagist so users will be able to install it with wp package install gravityforms/gravityformscli.

Related Issue

Closes: #4938

Testing

Testing Requirements

  • This needs to be tested in multiple browsers.
  • This needs to be tested with both Gravity and Orbital themes.
  • This needs to be tested with and without AJAX submission.
  • This needs accessibility testing.
  • This needs to be tested with different versions of PHP or database servers.
  • This needs to be tested with different versions of WordPress.
  • None of the above.

Testing Instructions

  1. Ensure the code changes here match those that have been suggested

Additional testing -
Once added to Packagist (post-merge), test by running the individual commands:

QA Tracking Spreadsheet

Checklist

  • I've checked the error log to ensure my code doesn't throw any errors.
  • I've added a changelog entry (if necessary) and updated the version number in 4 places, run npm install and committed package-lock.json (if necessary).
  • I have added a "Documentation Needed" label to this PR (if necessary).

@ibd21 ibd21 self-assigned this Mar 11, 2026
@ibd21 ibd21 marked this pull request as ready for review March 11, 2026 17:05
@ibd21 ibd21 requested a review from a team March 11, 2026 17:05
@SherifMesallam SherifMesallam self-requested a review March 16, 2026 17:07
Copy link
Contributor

@SherifMesallam SherifMesallam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good here, I just allowed myself to make some minor modifications, left comments to explain.


if ( ! class_exists( 'GFAddOn' ) ) {
return;
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a cleaner exit instead of wrapping the whole class into the check.

"gf entry",
"gf entry notification",
"gf license",
"gf tool"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add all the commands that are actually registered in cli.php

1. Wordpress 4.2+
2. Gravity Forms > 1.9.17.8
3. WP-CLI v1.0+
3. WP-CLI v2.5+
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need this as a minimum now

@ibd21
Copy link
Contributor Author

ibd21 commented Mar 17, 2026

@SherifMesallam -- thanks for jumping in here!! All the changes above look great, will get this merged in now.

@ibd21 ibd21 merged commit ac27362 into master Mar 17, 2026
@ibd21 ibd21 deleted the fix-4938-packagist-compatibility branch March 17, 2026 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants