-
Notifications
You must be signed in to change notification settings - Fork 4
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
Store Flow Changes (Support for restore flow) #1113
Conversation
Got the issue on IOS about the app tracking policy working on that |
@@ -36,14 +36,34 @@ class AppMethods { | |||
} | |||
|
|||
static Future<bool> isPlayStoreEnable() async { | |||
if(!Platform.isAndroid){ | |||
if (!Platform.isAndroid) { | |||
return false; | |||
} | |||
final isPlayVersion = sessionModel.isStoreVersion.value ?? false; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried testing these changes on Android by setting FORCE_PLAY_VERSION=true, and It looks like isPlayVersion always returns false. Are we setting storeVersion on the session model to BuildConfig.PLAY_VERSION
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For me, it is working, Could you confirm that your is not a pro?
I did some testing (by manually setting
This may be unrelated to this PR, but I wonder if the pro server may not be updating users to have verified email addresses after they enter a valid confirmation code (unless I'm getting that error for some other reason?). |
This is correct, You get this error only when you enter an expired code while verification. Can you try again and see if that's an issue, it should work. |
@atavism Could you review this? |
@atavism Could you please give this PR spin? I like to merge this if all okay |
No description provided.