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.

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

Add option to supply default path extension #456

Merged
merged 1 commit into from
Sep 27, 2016
Merged

Add option to supply default path extension #456

merged 1 commit into from
Sep 27, 2016

Conversation

boyvanamstel
Copy link

Especially useful when using Kingfisher on macOS and wanting to add
drag-drop. Most web input fields won't accept files without a path
extension (like .jpg).

Currently only supports a single default path extension. Could later
be expanded to include a path extension per image.

Especially useful when using Kingfisher on macOS and wanting to add
drag-drop. Most web input fields won't accept files without a path
extension (like .jpg).

Currently only supports a single default path extension. Could later
be expanded to include a path extension per image.
@codecov-io
Copy link

codecov-io commented Sep 27, 2016

Current coverage is 74.40% (diff: 100%)

Merging #456 into master will increase coverage by 0.25%

@@             master       #456   diff @@
==========================================
  Files            21         21          
  Lines          2194       2196     +2   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits           1627       1634     +7   
+ Misses          567        562     -5   
  Partials          0          0          

Powered by Codecov. Last update 1118bd0...2b6b307

@onevcat
Copy link
Owner

onevcat commented Sep 27, 2016

LGTM

@onevcat onevcat merged commit 4131bd6 into onevcat:master Sep 27, 2016
rain2540 pushed a commit to rain2540/Kingfisher that referenced this pull request Feb 28, 2017
Add option to supply default path extension
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants