Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Upgraded] For paperclip 3.1.2, with S3 fixes, and generic paperclip versions #69

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

[Upgraded] For paperclip 3.1.2, with S3 fixes, and generic paperclip versions #69

wants to merge 4 commits into from

Commits on Mar 20, 2012

  1. [Fix] most_appropiate_url changed to be part of Paperclip::UrlGenerat… 

    …or, tested across rails versions and added Rails 3.2 to test suite. Paperclip version bumped to 2.7.

  - Main issue - `attachment`_updated_at was not being updated, new Paperclip (> 2.4.5) assigns the updated_at of the `attachment`, when S3 took too long, or the queue takes longer than a second to pick the image up, the `attachment`_updated_at and instance.updated_at get out of sync, and the url generated (with :updated_at in the hash_data - default), will not find the newly created images.
    @tommeier
    tommeier committed Mar 20, 2012
    Configuration menu
    Copy the full SHA
    98a8b9e View commit details
    Browse the repository at this point in the history

Commits on Jul 2, 2012

  1. [Upgrade] Allow paperclip 3.x to work, tested with 3.1.2

    @tommeier
    tommeier committed Jul 2, 2012
    Configuration menu
    Copy the full SHA
    4e07528 View commit details
    Browse the repository at this point in the history

  2. [Update] Unnecesary to mark post_processing as false, because papercl… 

    …ip does this in reprocess, test updated to ensure outcome
    @tommeier
    tommeier committed Jul 2, 2012
    Configuration menu
    Copy the full SHA
    c7ce367 View commit details
    Browse the repository at this point in the history

  3. Removed commented line

    @tommeier
    tommeier committed Jul 2, 2012
    Configuration menu
    Copy the full SHA
    c23a537 View commit details
    Browse the repository at this point in the history