Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
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.
Comment | File | Size | Author |
---|---|---|---|
#4 | additional_status-2271881-4.patch | 4.74 KB | bkonetzny |
Comments
Comment #1
drnikki CreditAttribution: drnikki commentedComment #2
drnikki CreditAttribution: drnikki commentedUpdated paths.
Comment #3
drnikki CreditAttribution: drnikki commentedComment #4
bkonetzny CreditAttribution: bkonetzny commentedUpdated the patch to show the configured bucket name or link to the configuration page, if no bucket is configured.
Comment #5
deviantintegral CreditAttribution: deviantintegral at Lullabot for NBCUniversal commentedI 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.
Comment #6
deviantintegral CreditAttribution: deviantintegral at Lullabot for NBCUniversal commented