Skip to content

fix(ios): fix path to PrivacyInfo.xcprivacy #98

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

Conversation

slobodanmilinkovic
Copy link
Contributor

🛠️ What this PR does

This PR fixes an Xcode warning related to a missing PrivacyInfo.xcprivacy file:

Invalid Resource 'PrivacyInfo.xcprivacy': File not found.

While this warning does not prevent the build from succeeding, it does show up in Xcode under "All Issues" and can be confusing or concerning to developers.

✅ Fix

Corrected the path to PrivacyInfo.xcprivacy so that Xcode can locate it properly in the Resources folder. This eliminates unnecessary warnings in Xcode.

Let me know if you'd like any changes — and thank you for maintaining this plugin!

Copy link
Collaborator

@diegotori diegotori left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@slobodanmilinkovic
Copy link
Contributor Author

@diegotori Thanks for the approval! I don’t have merge permissions, so could you please merge this when you get a chance? Appreciate it! 🙏 Just to note, these 2 checks failing aren’t related to the changes I made 🙈

@diegotori
Copy link
Collaborator

@diegotori Thanks for the approval! I don’t have merge permissions, so could you please merge this when you get a chance? Appreciate it! 🙏 Just to note, these 2 checks failing aren’t related to the changes I made 🙈

Don't worry about the failing lint checks, since the workflow needs to be adjusted so that it performs them on older versions of Flutter. I'll merge and deploy very shortly.

@diegotori diegotori merged commit 8337681 into fluttercommunity:main Apr 24, 2025
12 of 14 checks passed
diegotori added a commit that referenced this pull request Apr 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants