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.

Sign up for GitHub

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

Jump to bottom

fix: remove tracking domain url from Privacy Manifest file in v2 #514

Merged
merged 2 commits into from
Apr 24, 2024
Merged

fix: remove tracking domain url from Privacy Manifest file in v2 #514

merged 2 commits into from
Apr 24, 2024

Conversation

1abhishekpandey
Copy link
Contributor

Description

  • We've decided to remove the tracking domain URL from the Privacy Manifest file in iOS-v2. This change aims to resolve the error that occurs when a request to our tracking domain gets blocked after the user denies tracking permission.

@1abhishekpandey 1abhishekpandey self-assigned this Apr 23, 2024
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Apr 23, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@1abhishekpandey 1abhishekpandey marked this pull request as ready for review April 23, 2024 15:38
@1abhishekpandey 1abhishekpandey requested a review from a team as a code owner April 23, 2024 15:38
@1abhishekpandey 1abhishekpandey merged commit 533e8e1 into develop-v2 Apr 24, 2024
17 checks passed
@devops-github-rudderstack devops-github-rudderstack mentioned this pull request Apr 24, 2024
Closed
@1abhishekpandey 1abhishekpandey mentioned this pull request Apr 24, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ChryssaAliferi ChryssaAliferi ChryssaAliferi approved these changes

Assignees

@1abhishekpandey 1abhishekpandey

Labels
Ready for review v2
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@1abhishekpandey @ChryssaAliferi