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

Fix #97 Android 10 screenshot sorted incorrectly #98

Merged
merged 1 commit into from
Nov 7, 2019

Conversation

scarlac
Copy link
Contributor

@scarlac scarlac commented Oct 24, 2019

Fix outlined in #97

Summary

Screenshots taken in Android 10 are no longer tagged with DATE_TAKEN but only DATE_ADDED. This causes screenshots taken in Android 10 to appear at the bottom of a potentially very long list.

Test Plan

I have no solid test plan. I've tested it locally.

What's required for testing (prerequisites)?

A photo library with at least 10 items, to clearly see how screenshots are shown last.

What are the steps to reproduce (after prerequisites)?

See issue

Compatibility

OS Implemented
iOS
Android

Checklist

  • I have tested this on a device and a simulator
  • I added the documentation in README.md
  • I mentioned this change in CHANGELOG.md
  • I updated the typed files (TS and Flow)
  • I added a sample use of the API in the example project (example/App.js)

@bartolkaruza bartolkaruza merged commit 902abd4 into react-native-cameraroll:master Nov 7, 2019
bartolkaruza pushed a commit that referenced this pull request Nov 7, 2019
# [1.3.0](v1.2.1...v1.3.0) (2019-11-07)

### Bug Fixes

* **example:** remove flow parsing from example ([452fd7b](452fd7b))
* **lib:** Android 10 screenshot sorted incorrectly ([#98](#98)) ([902abd4](902abd4)), closes [#97](#97)
* **lib:** Fix warning for deprecated RCTBridge interface for RCTImageLoader ([#87](#87)) ([c0a132d](c0a132d))

### Features

* **lib:** Moved deletePhotos to use new PHAsset API and added an implementation for Android ([#69](#69)) ([7850dd5](7850dd5))
@bartolkaruza
Copy link
Collaborator

🎉 This PR is included in version 1.3.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@scarlac scarlac deleted the patch-1 branch December 11, 2019 19:33
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.

2 participants