Skip to content

fix(ad-hoc): Support screen rotation with POCardTokenizationViewComponent - #403

Merged
vitalii-vanziak-cko merged 6 commits into
masterfrom
fix/ad-hoc/support-card-view-rotation
Oct 22, 2026
Merged

vitalii-vanziak-cko merged 6 commits into
masterfrom
fix/ad-hoc/support-card-view-rotation

Conversation

@vitalii-vanziak-cko

@vitalii-vanziak-cko vitalii-vanziak-cko commented Oct 22, 2026 •

Copy link
Copy Markdown
Collaborator

Supporting screen rotation with POCardTokenizationViewComponent:

  • Ensures that component uses the same eventDispatcher instance as interactor, in case component was recreated during rotation.
  • Using correct lifecycleOwner and coroutineScope.
  • Do not lock orientation on POCustomTabAuthorizationActivity and POWebViewAuthorizationActivity, instances will not be recreated on rotation because of configChanges flags in manifest.

Note:
Merchant owner activity needs to specify flags in manifest:
android:configChanges="orientation|screenSize"
This will prevent recreation of owner Activity/Fragment, it's scope and bound UI components.

@andrii-vysotskyi-cko andrii-vysotskyi-cko left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

There seems to be a similar root cause: POEventDispatcher is a replay-0 SharedFlow and nothing re-sends pending requests or the current state when a new subscriber attaches after rotation.

@vitalii-vanziak-cko
vitalii-vanziak-cko merged commit 67fd0c4 into master Oct 22, 2026
3 checks passed
@vitalii-vanziak-cko
vitalii-vanziak-cko deleted the fix/ad-hoc/support-card-view-rotation branch October 22, 2026 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants