GravityKit / GravityExport-Lite

GravityExport Lite. Export all Gravity Forms entries to Excel (.xlsx) via a secret (shareable) url.
https://gfexcel.com
GNU General Public License v2.0
12 stars 8 forks source link

Release 2.1.0 (Strauss) #181

Closed doekenorg closed 11 months ago

doekenorg commented 1 year ago

This PR adds Strauss + the necessary configuration. This also Resolves #165.

Upsides:

Downsides:

The changes are pretty easily fixed though.

It might be possible to alias the old class to the new names, in case those classes don't exist. This is a runtime check, and would avoid initial errors on plugins. Giving us time to fix the plugins.

For release:

Release contains:

Todo:

For review also check:

For testing:

Attached is a test-plugin. If this plugin is enabled, it will trigger the loading of a newer version of league/container. To fix the break, just comment out the single line in the a-test-plugin.php file.

  1. This build should not cause the error once the test plugin is activated.
  2. The pro version will not work if the test plugin and this build are both enabled
  3. Disabling the test plugin should fix the pro version from working.
  4. The build from https://github.com/GravityKit/GravityExport/pull/87 should work with both the test plugin and this build together.

Please test thoroughly if all features still work. Think: Xlsx, CSV, PDF. Filter feeds, Save dropbox feeds.

a-test-plugin.zip

doekenorg commented 12 months ago

I've moved strauss.phar along with a composer.json to the build folder. In de src folder, there is now a composer.json which copies the src folder to the vendor folder, so that strauss can do it's magic on that too. From that point we check in gfexcel.php if the build-autoloader exists. It will use the fixed source files, making the plugin wp.org proof; whilst also being installable on a bedrock project.

Todo:

mrcasual commented 11 months ago

@doekenorg, feel free to merge if this is done. The PR works.