Non-technical users would benefit from slightly more robust status report notifications. If something is missing from s3 settings, we can notify on the status report rather than have it only surface when a user tries to read or write.

It would be nice in the future to test write-ability if amazons3_folder is provided.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

drnikki’s picture

drnikki’s picture

Updated paths.

drnikki’s picture

FileSize
4.29 KB
bkonetzny’s picture

Status: Active » Needs review
FileSize
4.74 KB

Updated the patch to show the configured bucket name or link to the configuration page, if no bucket is configured.

deviantintegral’s picture

Version: 7.x-1.x-dev » 7.x-2.x-dev

I ported this to the 7.x-2.x branch over at https://github.com/justafish/drupal_amazons3/pull/21

I also added test coverage for using Drupal variables, and fixed issues where someone doesn't run composer install but enables the S3 module.

deviantintegral’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.