Skip to content

[DX-2700] fix: android login and connect pkce not working if minify is enabled in sample app #173

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

Merged
merged 1 commit into from
Feb 25, 2024

Conversation

nattb8
Copy link
Collaborator

@nattb8 nattb8 commented Feb 25, 2024

Summary

As pointed out by #172, the LoginPKCE and ConnectImxPKCE functions do not work on Android if Minify is enabled. This PR added an example demonstrating how to configure Proguard with Minify enabled in the sample game.

Also, fixed the sample application to log out using the appropriate method if the login or connection attempt fails.

Customer Impact

If you enable Minify in your game, please follow the instructions in our documentation here.

Other things to consider:

  • Sample app is updated with new SDK changes
  • Updated public documentation with new SDK changes (Immutable X and Immutable zkEVM)
  • Sample game is updated with new SDK changes
  • Replied to GitHub issues

@nattb8 nattb8 requested a review from a team February 25, 2024 22:05
@nattb8 nattb8 changed the title [DX-2700] fix: android login and connect pkce not working if minify is enabled [DX-2700] fix: android login and connect pkce not working if minify is enabled in sample app Feb 25, 2024
@nattb8 nattb8 force-pushed the fix/android-proguard branch from 5f67a09 to 183b45d Compare February 25, 2024 22:07
@nattb8 nattb8 merged commit a9b85e9 into main Feb 25, 2024
@nattb8 nattb8 deleted the fix/android-proguard branch February 25, 2024 22:13
@github-actions github-actions bot added the fix label Feb 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants