WordPress-iOS icon indicating copy to clipboard operation
WordPress-iOS copied to clipboard
verified publisher icon wordpress-mobile

Compliance popover fails to save settings and dismiss

Open kean opened this issue 1 year ago • 3 comments

Expected behavior

  • The popover is dismissed

Actual behavior

  • The popover is not dismissed

Workaround: Tap "Go to Settings"

Steps to reproduce the behavior

  • Install the app for the first time
  • Login with a country with GDPR
  • Add a self-hosted site
  • Verify that a compliance popover is shown
  • Tap "Save Settings"

https://github.com/wordpress-mobile/WordPress-iOS/assets/1567433/6ca65dd6-d239-418a-bbed-db03ee13cfce

Tested on [device], iOS [version], Jetpack iOS / WordPress iOS [version]

trunk

Affects self-hosted sites in countries with GDPR.

kean avatar Jul 11 '24 12:07 kean

Thanks for reporting! 👍

dangermattic avatar Jul 11 '24 12:07 dangermattic

RCA

CompliancePopoverViewModel returns without doing anything if the wp.com accountId is not present: https://github.com/wordpress-mobile/WordPress-iOS/blob/2a20648fe23bdc50c279c793a166b2b883597a8f/WordPress/Classes/ViewRelated/EEUUSCompliance/CompliancePopoverViewModel.swift#L49

kean avatar Jul 11 '24 12:07 kean

I've just encountered this, I'm not sure if it's related but once I managed to close this modal, the suggestion to "Save your login credentials" system modal was behind it.

geriux avatar Aug 13 '24 12:08 geriux