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

Feature: add new context option to use request image data instead of request image API #7

Merged
merged 4 commits into from
Apr 8, 2019

Conversation

dreampiggy
Copy link
Collaborator

This PR introduce a context option SDWebImageContextPhotosRequestImageData, to always using the requestImageDataForAsset:options:resultHandler: API for Photos framework.

This is useful for user, who need the raw image data, and have their image processing code. The default behavior only using this for animated images, which may not satisfy the use case.

…orAsset` API instead of `requestImageForAsset`, this may help for user who need raw image data and custom processing pipeline.
@dreampiggy dreampiggy requested a review from a team April 8, 2019 05:18
@codecov
Copy link

codecov bot commented Apr 8, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@82df99e). Click here to learn what that means.
The diff coverage is 91.66%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master       #7   +/-   ##
=========================================
  Coverage          ?   63.38%           
=========================================
  Files             ?        3           
  Lines             ?      254           
  Branches          ?        0           
=========================================
  Hits              ?      161           
  Misses            ?       93           
  Partials          ?        0
Flag Coverage Δ
#iOS 63.38% <91.66%> (?)
Impacted Files Coverage Δ
...ImagePhotosPlugin/Classes/SDWebImagePhotosLoader.m 59.4% <91.66%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 82df99e...efe1960. Read the comment docs.

@dreampiggy dreampiggy merged commit b391207 into master Apr 8, 2019
@dreampiggy dreampiggy deleted the feature_request_image_data branch April 8, 2019 10:41
@dreampiggy dreampiggy changed the title Feature: request image data context option to always request image data Feature: add new context option to use request image data instead of request image API Apr 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant