-
-
Notifications
You must be signed in to change notification settings - Fork 235
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.
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
Allow moderation to potentially change photos. #2291
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2291 +/- ##
==========================================
- Coverage 80.24% 80.16% -0.09%
==========================================
Files 187 187
Lines 12039 12035 -4
Branches 2224 2224
==========================================
- Hits 9661 9648 -13
- Misses 1617 1630 +13
+ Partials 761 757 -4
Continue to review full report at Codecov.
|
3b341b6
to
d7d6f99
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small comments but this looks good.
$c->forward('/photo/process_photo'); | ||
#if ( my $photo_error = delete $c->stash->{photo_error} ) { | ||
# What to do here? | ||
#} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be resolved?
Some fixups for the error, test, and then to prevent clashes with the update photo upload (both JS only looking for one form, and server not putting the problem photos in the update photo upload preview). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this could do with a link to an example of how to add it somewhere because it's not entirely obvious. But that's a small point. Looks good otherwise.
70b83e9
to
1e5b99f
Compare
As well as the original form using #form_photo, add Dropzone to any form with a .js-photo-label in it.
1e5b99f
to
c73e8b5
Compare
Lets a cobrand provide a standard photo upload form (as on new report/update) in the report display so moderation can add/remove photos to a report, not just show/hide.