WP S3 plugin needs full package with vendor folder

[jms-pantheon]

Summary

AWS S3 Setup for WordPress - Download instructions were unclear.

[pantheon-decoupled]

⚡ Deployed with Pantheon Decoupled

This build was successfully deployed with Pantheon. You can track the build logs here.

👀 Preview: https://pr-9257-documentation.appa.pantheon.site
🛠️ Manage in Pantheon: https://dashboard.pantheon.io/site/2b30153f-e8b1-4427-b076-6109e704ba5d/overview

[rachelwhitton]

assigning to @jazzsequence to consider testing with the latest version of the plugin and updating docs

[jms-pantheon]

Reviewing this now, it looks like that plugin is removing the manual install. Perhaps we should just refer people to that project's readme for install instructions

[jazzsequence]

Looking at the HM repo now, and in particular the line:

Composer's autoloader must be loaded before S3 Uploads is loaded.

I'm thinking that it's not possible to install S3 Uploads without composer. We should call that out specifically in the opening paragraph as a requirement. I also don't think that WP Offload Media requires a license (I am using it on my personal site without one), but it's required if you want the urls to match your domain (rather than the S3 bucket or DO space or whatever). So maybe we can lean heavier into using that plugin instead.

Given we just link out to WP Offload Media for docs/instruction there, I think we should probably do the same for S3 Uploads (maybe with a note that WP-CLI commands in HM's docs must be wrapped in Terminus syntax for use on a Pantheon site.

[jazzsequence]

@jms-pantheon I pushed up some changes that remove the detailed instructions for S3 Uploads in favor of just linking over to their README. I also tweaked some of the language around Offload Media and noted that S3 Uploads requires Composer. Given it's a plugin that we don't maintain, it seems like a better practice to link out to their docs rather than trying to recreate them. Lemme know what you think and we can get this merged.

[pantheon-decoupled]

⚡ Deployed with Pantheon Decoupled

This build was successfully deployed with Pantheon. You can track the build logs here.

👀 Preview: https://pr-9257-documentation.appa.pantheon.site
🛠️ Manage in Pantheon: https://dashboard.pantheon.io/site/2b30153f-e8b1-4427-b076-6109e704ba5d/overview

[jms-pantheon]

Thanks @jazzsequence that looks like a good move

[jazzsequence]

Merging!