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

Use Paperclip for attachments #88

Closed
peakpg opened this issue May 6, 2011 · 2 comments
Closed

Use Paperclip for attachments #88

peakpg opened this issue May 6, 2011 · 2 comments
Milestone
3.5

Comments

@peakpg
Copy link
Collaborator

peakpg commented May 6, 2011

Imported from lighthouse. Original ticket at: http://browsermedia.lighthouseapp.com/projects/28481/tickets/342. Created by Tyler Rick - 2010-04-23 18:20:43 UTC

It looks like you're considering adding Paperclip support already ( #250, #207) and attachments clearly need some overhauling ( #28), so this is probably unnecessary, but I didn't see a ticket for this and thought it might be nice to have one...

The main benefits I see for using Paperclip are:

  • Leverage an existing open-source project and benefit from improvements/bug fixes
  • Developer-users coming to BrowserCMS are fairly likely to have used Paperclip and would benefit from a familiar API
  • Support for multiple versions (i.e., thumbnail and medium version for image attachments)
  • Post-processors
  • Flexible validation abilities
This was referenced Mar 1, 2012
Closed
Closed
peakpg added a commit that referenced this issue Apr 2, 2012
@peakpg
[#88] Use Paperclip for attachments
650652a 
* Bring in initial Scenarios from Assets
* Add test files
* Add new fields to attachments table
@peakpg peakpg mentioned this issue Apr 19, 2012
Closed
@peakpg
Copy link
Collaborator Author

peakpg commented May 9, 2012

This is done.

@peakpg peakpg closed this as completed May 9, 2012
peakpg added a commit that referenced this issue May 9, 2012
@peakpg
[#88] Improved Attachments API
ecbdc6f 
Can have multiple attachments, using Paperclip rather than our custom hand rolled version.
peakpg added a commit that referenced this issue May 10, 2012
@peakpg
[#88] Use Paperclip for attachments
a14b177 
* Bring in initial Scenarios from Assets
* Add test files
* Add new fields to attachments table
peakpg added a commit that referenced this issue May 10, 2012
@peakpg
[#88] Improved Attachments API
801447a 
Can have multiple attachments, using Paperclip rather than our custom hand rolled version.
@courtsimas
Copy link

any ETA for supporting paperclip options such as before_post_process, or even the ability to use delayed jobs for post processing? (delayed_paperclip gem doesn't work with browsercms unfortunately)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
3.5
Development

No branches or pull requests
2 participants
@peakpg @courtsimas