-
Notifications
You must be signed in to change notification settings - Fork 698
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 issue 1367 #1369
base: master
Are you sure you want to change the base?
Fix issue 1367 #1369
Conversation
@miguelpruivo Please look at the Issue for more details. |
Is this still an issue with the latest iOS? |
I have an iOS 17.1.1 and im getting an error when i choose live photo. With this PR, there is no error 👍🏻 |
Is this fix going to be approved? I have the exact same issue. |
Please rebase and update the version/changelog accordingly. |
Do we have any news about this? @GabrielAraujo @miguelpruivo is there a chance that this PR could be merged into the main repository sooner rather than later? 🤔 |
Would be awesome to see this PR merge! |
Please merge this PR as soon as possible? @miguelpruivo |
We rebase & update the changelog the PR with main here: |
Any update on this? I am getting the error more and more. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please rebase to main and I'll merge it right away. Sorry for the delay.
@miguelpruivo Rebased! Please merge this asap |
@miguelpruivo any chance you can merge this PR? It's been open for more than a year 🙏 |
@GabrielAraujo any chance you can rebase the branch and update the version number? |
@matiasmelendi do you have permissions to merge this? It has been a full year since we started this discussion but the maintainers never merge it and we need to keep constantly rebasing it ...... |
I don't but I promise I will do my best to chase @miguelpruivo to re-review the PR and merge it. I really need this to be merged and released 🙏 |
Issue 1367
On iOS 17 the UTI type public.item is not properly working for live photos that are syncing with iCloud (download or upload).
Using media specific UTI types seemed to have fixed the problem but still any thoughts on the approach are welcome so we can adjust on how we handle the issue.