-
Notifications
You must be signed in to change notification settings - Fork 224
Fixes #24: Allow images to be uploaded when creating a new bug report. #679
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
fd93070
Don't swallow the details of the exception.
a8d9c09
Issue #24. Add an upload blueprint + route.
6cce652
Issue #24. Serve images from Flask -- but only for local development.
df49e0b
Issue #24. Update config.py.example with new config settings.
59b51ed
Issue #24. Hook up image uploading with bug reporting.
35dd685
Issue #24. Tweak upload form label and add accept attr to input[type=…
0300844
Issue #24. Be less liberal and only accept: jpeg, png, gif and bmp.
174636d
Issue #24. Display errors in template for image uploading.
4d2c359
Issue #24. Only try to add an image to the issue... if there's an image.
6f593c6
Issue #24. Clean up the BugForm consructor object.
f521304
Issue #24. Show loader when uploading an image.
1cb0588
Issue #24 - added icon upload
magsout c3c9ec0
Issue #24. Add note about uploads config error.
d39f21d
Issues #24 - added style and icon for upload button
magsout 47ffe9c
Issue #24: Add pos: relative to wc-Form-upload to prevent pos: abs in…
d9e7e59
Issue #24: Add missing url form error class.
8457d9e
Issue #24: Use Flask-WTF for crsf + File upload validation.
d70f002
Issue #24: Add unit tests for uploads.
fa895e1
Issue #24: Do clientside validation for image type.
c1c70eb
Issue #24: Split up form validation tests and add one for valid images.
7a26480
Issue #24. Remove unused method import.
b46af3d
Issue #24. Re-format how we print ImportErrors to the console.
2162300
Issue #24. Reformat some for loops.
abbbd9a
Issue #24. Remove unused import.
c8dc773
Issue #24: Import TestingFileStorage (oops)
ef65a53
Issue #24: Remove extra uploads/ dir for localhost.
bb9c875
Issue #24. Obfuscate the uploaded filename.
bd14e70
Issue #24. Save uploads to a year/month/ namespaced location.
3790df1
Issue #24. Update privacy policy to mention image uploads.
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Issue #24. Save uploads to a year/month/ namespaced location.
- Loading branch information
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This comment was marked as abuse.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.