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

Media #80

Merged
merged 22 commits into from
Sep 18, 2014
Merged

Media #80

merged 22 commits into from
Sep 18, 2014

Conversation

andersoncardoso
Copy link
Member

  • add pictures to object

@andersoncardoso
move avatar upload to own action + reload image when uploaded
a0b00be
@andersoncardoso
flash message when upload is sucessfull
a56e5b2
@andersoncardoso
add picture uploader and model
50af9f0
@andersoncardoso
initial pictures uploader + changes to uploader to be more extensible
12f0c9b
@andersoncardoso
add uploaded images to gallery
8659cd6
@andersoncardoso
change picture_upload to be a embeded concern
decf4c9
@andersoncardoso
fix picture model + add polymorphic relation
775325e
@andersoncardoso
save pictures associated to objects
d13a8f4
@andersoncardoso
load associated pictures
5f74042
@andersoncardoso
improve gallery
94e02a1
@andersoncardoso
show pictures + modal component
53abc90
@andersoncardoso
show images modal
28d10e9
@andersoncardoso
picture show with description field and action buttons
7525aef
@andersoncardoso
fix picture show + start added pictures components
02444a3
@andersoncardoso
picture description
7d5b0be
@andersoncardoso
remove image
6417668
@andersoncardoso
i18n
ae3f7d3
@andersoncardoso
show gallery
cf4e28d
@andersoncardoso
extract thumbs list to partial
bfcabf3
@andersoncardoso
test upload and show on pictures controller
efb9091
@andersoncardoso
more specs + change picture column to avoid bad name (object)
cf8f573
@andersoncardoso
more specs
d30d554
@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) when pulling d30d554 on media into a9c2de0 on master.

LuizArmesto added a commit that referenced this pull request Sep 18, 2014
@LuizArmesto
Merge pull request #80 from it3s/media
db9f650 
Media
@LuizArmesto LuizArmesto merged commit db9f650 into master Sep 18, 2014
@LuizArmesto LuizArmesto deleted the media branch September 18, 2014 23:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@andersoncardoso @coveralls @LuizArmesto