Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Merge branch 'master' into activestorage-guide | Jeffrey Guenther | 2017-12-14 | 9 | -20/+25 |
|\ | |||||
| * | Merge pull request #31434 from olivierlacan/boot-feedback | Sean Griffin | 2017-12-14 | 1 | -0/+4 |
| |\ | | | | | | | Provide instant feedback when booting Rails | ||||
| | * | Provide instant feedback when booting Rails | Olivier Lacan | 2017-12-13 | 1 | -0/+4 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I've noticed during pair/mob programming sessions with peers that despite the speed boosts provided by Bootsnap and Spring, there is a noticeable latency between firing a bin/rails server command and any feedback being provided to the console. Depending on the size of the application this lack of feedback can make it seem like something is wrong when Rails is simply busy initializing. This change may seem gratuitous but by just printing one line to STDOUT we're giving a clear signal to the Rails user that their command has been received and that Rails is indeed booting. It almost imperciptibly makes Rails feel more responsive. Sure the code doesn't look very fancy but there's no other appropriate place I could think of putting it than boot.rb. Compare these two GIFs of booting without and with this change: Before: ![Without Boot Feedback](https://user-images.githubusercontent.com/65950/33964140-721041fc-e025-11e7-9b25-9d839ce92977.gif) After: ![With Boot Feedback](https://user-images.githubusercontent.com/65950/33964151-79e12f86-e025-11e7-93e9-7a75c70d408f.gif) | ||||
| * | | Enable `Layout/LeadingCommentSpace` to not allow cosmetic changes in the future | Ryuta Kamizono | 2017-12-14 | 1 | -4/+0 |
| |/ | | | | | | | Follow up of #31432. | ||||
| * | Expose Active Storage routes | Mehmet Emin INAC | 2017-12-13 | 1 | -6/+6 |
| | | |||||
| * | Exclude ActiveStorage::Filename{#parameters,::Parameters} from API docs [ci ↵ | George Claghorn | 2017-12-13 | 2 | -2/+2 |
| | | | | | | | | skip] | ||||
| * | Merge pull request #31391 from ↵ | Eileen M. Uchitelle | 2017-12-13 | 1 | -1/+5 |
| |\ | | | | | | | | | | | | | 5t111111/fix-active_storage-installation-failure-in-engine Fix active_storage installation failure when in engine | ||||
| | * | Invoke app-prefixed active storage task when in engine | Hirofumi Wakasugi | 2017-12-13 | 1 | -1/+5 |
| | | | |||||
| * | | Fix optimizing GIF variants using mogrify's -layers option | George Claghorn | 2017-12-11 | 1 | -5/+7 |
| | | | |||||
| * | | Revert "Invoke mogrify once when transforming an image" | George Claghorn | 2017-12-11 | 1 | -8/+6 |
| | | | | | | | | | | | | This reverts commit a80f81af055f02bf4625c90470aa90441cf6fc24. | ||||
| * | | Merge pull request #31399 from yhirano55/return_nil_if_raise_http_error | George Claghorn | 2017-12-11 | 1 | -1/+1 |
| |\ \ | | | | | | | | | Return `nil` instead of `false` if raise `Azure::Core::Http::HTTPError` | ||||
| | * | | Return `nil` instead of `false` if raise `Azure::Core::Http::HTTPError` | Yoshiyuki Hirano | 2017-12-12 | 1 | -1/+1 |
| | |/ | | | | | | | | | | | | | | | | | | | * If it raise error `Azure::Core::Http::HTTPError`, return `nil` instead of `false` in `ActiveStorage::Service::AzureStorageService#delete`. * Other services behave as same as this. | ||||
| * | | Invoke mogrify once when transforming an image | George Claghorn | 2017-12-11 | 1 | -6/+8 |
| | | | | | | | | | | | | Execute a single mogrify command with multiple options rather than one command per option. Permit the use of all mogrify options, not just the ones that fall through to MiniMagick::Image#method_missing. | ||||
| * | | `webpack` is assigned but never used in webpack.config.js | Yoshiyuki Hirano | 2017-12-11 | 1 | -1/+0 |
| |/ | | | | | | | * Removed webpack const, so it is assigned but never used in webpack.config.js. | ||||
* | | Merge branch 'master' into activestorage-guide | Jeffrey Guenther | 2017-12-08 | 17 | -39/+179 |
|\| | |||||
| * | Swap raw video width and height if angle is 90 or 270 degrees | George Claghorn | 2017-12-08 | 2 | -3/+15 |
| | | |||||
| * | Fix customizing Content-Type via GCS service URLs | George Claghorn | 2017-12-07 | 2 | -1/+21 |
| | | |||||
| * | Fix instrumention name: delete_prefixed like the others. | Kasper Timm Hansen | 2017-12-03 | 1 | -1/+1 |
| | | |||||
| * | Purge variants with their blobs | George Claghorn | 2017-12-02 | 10 | -31/+107 |
| | | |||||
| * | Make ASt previewer/analyzer binary paths configurable | George Claghorn | 2017-12-01 | 4 | -4/+32 |
| | | |||||
| * | Add ActiveStorage::Previewer#logger to match ActiveStorage::Analyzer#logger | George Claghorn | 2017-11-30 | 1 | -0/+4 |
| | | |||||
* | | Merge branch 'master' into activestorage-guide | Jeffrey Guenther | 2017-11-30 | 10 | -27/+44 |
|\| | |||||
| * | Use `credentials` instead of `keyfile` in GCS sevice | yuuji.yaginuma | 2017-11-29 | 2 | -1/+3 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | The `keyfile` was renamed to `credentials` in `google-cloud-storage` 1.8.0. https://github.com/GoogleCloudPlatform/google-cloud-ruby/blob/master/google-cloud-storage/CHANGELOG.md#180--2017-11-14 Although `keyfile` can still be used, but it looks like deprecate. https://github.com/GoogleCloudPlatform/google-cloud-ruby/blob/ddf7b2a856d676316525eb581c1a4cc83ca6097b/google-cloud-storage/lib/google/cloud/storage.rb#L589...L590 Therefore, I think that should use `credentials` in newly generated applications. Ref: https://github.com/GoogleCloudPlatform/google-cloud-ruby/issues/1802 | ||||
| * | Preparing for 5.2.0.beta2 release | Rafael Mendonça França | 2017-11-28 | 3 | -2/+9 |
| | | |||||
| * | Fix typos and add a few suggestions | Fatos Morina | 2017-11-28 | 1 | -0/+14 |
| | | |||||
| * | Include migration files in gem | yuuji.yaginuma | 2017-11-28 | 1 | -1/+1 |
| | | | | | | | | Fixes #31245 | ||||
| * | Preparing for 5.2.0.beta1 release | Rafael Mendonça França | 2017-11-27 | 3 | -2/+4 |
| | | |||||
| * | Update yarn lock | Rafael Mendonça França | 2017-11-27 | 1 | -10/+0 |
| | | |||||
| * | Avoid connecting to GCS during app boot | George Claghorn | 2017-11-23 | 2 | -12/+14 |
| | | |||||
| * | Fix loading ActiveStorage::DiskController when CSRF protection is disabled ↵ | George Claghorn | 2017-11-21 | 1 | -1/+1 |
| | | | | | | | | by default | ||||
* | | Merge branch 'master' into activestorage-guide | Jeffrey Guenther | 2017-11-21 | 1 | -10/+10 |
|\| | |||||
| * | Fix that some ASt route helpers silently discarded options | George Claghorn | 2017-11-21 | 1 | -10/+10 |
| | | |||||
* | | Merge branch 'master' into activestorage-guide | Jeffrey Guenther | 2017-11-20 | 9 | -17/+169 |
|\| | |||||
| * | Provide attachment writers | George Claghorn | 2017-11-20 | 2 | -0/+40 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | Permit creating a record and attaching files in a single step. # Before: User.create!(user_params.except(:avatar)).tap do |user| user.avatar.attach(user_params[:avatar]) end # After: User.create!(user_params) [Yoshiyuki Hirano & George Claghorn] | ||||
| * | Load 5.2 defaults in ASt dummy app | George Claghorn | 2017-11-20 | 1 | -1/+1 |
| | | |||||
| * | Fix direct uploads to local service | George Claghorn | 2017-11-20 | 3 | -0/+6 |
| | | | | | | | | | | | | | | | | Disable CSRF protection for ActiveStorage::DiskController#update. The local disk service is intended to imitate a third-party service like S3 or GCS, so we don't care where direct uploads originate: they’re authorized by signed tokens. Closes #30290. [Shinichi Maeshima & George Claghorn] | ||||
| * | Permit attaching files to new records | George Claghorn | 2017-11-19 | 3 | -5/+53 |
| | | | | | | | | Closes #31164. | ||||
| * | Fix ASt CI failure with rack-test 0.7.1 | Ryuta Kamizono | 2017-11-20 | 1 | -1/+1 |
| | | | | | | | | Due to https://github.com/rack-test/rack-test/commit/5fd3631078e7c73aaed7d4371f70fb2a79384be9. | ||||
| * | Introduce ActiveStorage::Attached::{One,Many}#detach | George Claghorn | 2017-11-14 | 3 | -7/+40 |
| | | |||||
| * | Preserve existing metadata when analyzing a blob | George Claghorn | 2017-11-13 | 3 | -3/+28 |
| | | | | | | | | Closes #31138. | ||||
* | | Merge branch 'master' into activestorage-guide | Jeffrey Guenther | 2017-11-09 | 8 | -23/+30 |
|\| | |||||
| * | Remove needless block parameter | George Claghorn | 2017-11-07 | 1 | -1/+1 |
| | | |||||
| * | Fix streaming downloads from S3/Azure Storage | George Claghorn | 2017-11-06 | 3 | -7/+17 |
| | | | | | | Closes #31073. | ||||
| * | Permit configuring Active Storage's job queue | George Claghorn | 2017-11-03 | 5 | -16/+13 |
| | | |||||
* | | Fixes typo in docs | Jeffrey Guenther | 2017-11-02 | 1 | -1/+1 |
|/ | |||||
* | Allow third-party previewers/analyzers to customize their tempdirs | George Claghorn | 2017-11-02 | 2 | -3/+10 |
| | |||||
* | Use the indicative mood consistently [ci skip] | George Claghorn | 2017-10-22 | 1 | -1/+1 |
| | |||||
* | Fix links [ci skip] | George Claghorn | 2017-10-22 | 1 | -2/+2 |
| | |||||
* | Extract metadata from images and videos | George Claghorn | 2017-10-22 | 25 | -37/+407 |
| | |||||
* | Merge pull request #30821 from GBH/active-storage-readme | Eileen M. Uchitelle | 2017-10-22 | 1 | -0/+4 |
|\ | | | | | ActiveStorage install note [skip ci] |