-
-
Notifications
You must be signed in to change notification settings - Fork 972
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 Privacy Manifest empty dict #580
Conversation
@TimOliver a release required merging this PR as it is a problem with everyone using this repo in RN projects. |
Thanks for that @swrobel! |
@ShivamKJJW A release is required... please? 😜 |
Thanks for quick merge. I don't have access to release a new version. You will have to do it. :) |
@ShivamKJJW So I don't actually use React Native. I'm in no rush to push a new release. 😇 Please be mindful in free open source comments to be polite and not demand things. 🙏 |
Oh! Please don't mind. I didn't take it in that way. Instead of demanding I was trying to drag your attention to this PR due to its urgency. Anyways got your concern now. Thanks. It will be really helpful for everyone if you can make a release. 🙏 |
@ShivamKJJW Ahh I see! No problem Shivam! Sorry I misunderstood then! I really appreciate that! I've just shipped a new release under 2.7.4. Please let me know if this resolved the issue or if it needs any follow-up! Thanks! |
Thank you so much @TimOliver. Sure, I will check it. |
Hi Tim, thanks for your work on TOCropViewController. |
Fixes #579