Skip to content
This repository was archived by the owner on Jun 9, 2023. It is now read-only.

Using MediaStore to notify downloaded media#35

Merged
Tunous merged 1 commit intoTunous:masterfrom
mvietri:fix/mediastore
Aug 11, 2019
Merged

Using MediaStore to notify downloaded media#35
Tunous merged 1 commit intoTunous:masterfrom
mvietri:fix/mediastore

Conversation

@mvietri
Copy link
Copy Markdown

@mvietri mvietri commented Aug 8, 2019

I had close my previous PR. In that one I've use a media scanner intent which will be deprecated soon. I now did re-apply those changes but using MediaStore.

I did only test on Pie. I'd like to get some feedback on previous API versions.

  • Test on Pie

  • Test on Nougat/Oreo

  • Test on other API versions

@mvietri
Copy link
Copy Markdown
Author

mvietri commented Aug 10, 2019

More info: Without this commit, tapping on the notification after downloading a image, on Pie, a toast comes up saying "Action Failed"

@Tunous Tunous added this to the 0.8.0 milestone Aug 10, 2019
@Tunous Tunous added the bug Something isn't working label Aug 10, 2019
@Tunous Tunous merged commit 757911c into Tunous:master Aug 11, 2019
@mvietri mvietri deleted the fix/mediastore branch August 11, 2019 11:51
mvietri pushed a commit to mvietri/Dawn that referenced this pull request Aug 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants