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.

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

(UMP) Result of ConsentInformation.PrivacyOptionsRequirementStatus is swapped on iOS #3007

Closed
marcelo-fanatee opened this issue Nov 9, 2023 · 1 comment
Assignees
Labels
P1 Priority issue. Plugin Issue with the Unity plugin.

Comments

@marcelo-fanatee
Copy link

[REQUIRED] Step 1: Describe your environment

  • Unity version: 2021.3.28f1
  • Google Mobile Ads Unity plugin version: 8.5.2, 8.5.3
  • Platform: iOS
  • Platform OS version: 16.6.1
  • Any specific devices issue occurs on: no
  • Mediation ad networks used, and their versions: none

[REQUIRED] Step 2: Describe the problem

Steps to reproduce:

Result of ConsentInformation.PrivacyOptionsRequirementStatus is swapped on iOS

  • initialize consent information
  • use ConsentForm.LoadAndShowConsentFormIfRequired
    • IF consent form was Required and did appear ConsentInformation.PrivacyOptionsRequirementStatus will be NotRequired
    • IF consent form was NotRequired ConsentInformation.PrivacyOptionsRequirementStatus will be Required

I believe the issue is that the enum for privacy options requirement status are different in ios plugin code and cs code
so value gets swapped when coming from native side

On android it works as expected

iOS plugin:

/// State values for whether the user needs to be provided a way to modify their privacy options.
typedef NS_ENUM(NSInteger, GADUPrivacyOptionsRequirementStatus) {
/// Requirement unknown.
kGADUPrivacyOptionsRequirementStatusUnknown = 0,
/// A way must be provided for the user to modify their privacy options.
kGADUPrivacyOptionsRequirementStatusRequired = 1,
/// User does not need to modify their privacy options. Either consent is not required, or the
/// consent type does not require modification.
kGADUPrivacyOptionsRequirementStatusNotRequired = 2,
};

Unity cs:

public enum PrivacyOptionsRequirementStatus
{
/// <summary>
/// Privacy options requirement status is unknown.
/// </summary>
Unknown = 0,
/// <summary>
/// Privacy options entry point is not required.
/// </summary>
NotRequired = 1,
/// <summary>
/// Privacy options entry point is required.
/// </summary>
Required = 2,
}

@ericleich
Copy link
Member

Thanks for the report and good catch. #3010 will be merged into the next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 Priority issue. Plugin Issue with the Unity plugin.
Projects
None yet
Development

No branches or pull requests

4 participants
@NVentimiglia @ericleich @marcelo-fanatee and others