search

uservoice / uservoice-android-sdk

UserVoice Android SDK
https://www.uservoice.com/mobile/
MIT License
117 stars 105 forks source link

Android N support #233

Open himanshurchhabra opened 8 years ago

himanshurchhabra commented 8 years ago

There seems to be some issue with the android N update. After entering the question/feedback when a user tries to enter the email address the cursor moves back to the title edit box and is never able to enter the email address hence can't submit.

Christophe668 commented 8 years ago

Same issue here

Christophe668 commented 8 years ago

Any update on this issue?

shdunning commented 8 years ago

Hey guys -- to help me narrow down the test cases can you provide the following details:

I've been unable to reproduce this running our demo app (UVDemo) with version 1.2.5 of our SDK on emulated Nexus 5X / 6P running Android N 7.0, 3.10.0 kernel.

Christophe668 commented 8 years ago

Hi,

It doesn't work on a physical Nexus 6P on Android 7.0 kernel 3.10.73 but it works (as you said) on emulators (Genymotion Nexus 6P 7.0 PREVIEW)

Basically, I'm not doing anything complicated, I'm just calling this in my application class :

Config config = new Config("..."); UserVoice.init(config, this);

and

UserVoice.launchContactUs(activity); when I need it.

shdunning commented 8 years ago

Thanks for the info @Christophe668. Can you try rebuilding with the changes in my tracking branch?

cd <local_repo>; git checkout -b SD_androidN origin/SD_androidN

I've made some changes to update dependencies to use the latest Android SDK support / build libraries.

In related news, I'm still unable to repro. We installed the UVDemo app this morning on a Nexus 5X running Android 7 and successfully submitted feedback.

Christophe668 commented 8 years ago

I tried on a Nexus 6P and a Nexus 5X on Android 7.0 but it doesn't seems to fix the problem unfortunately. I can't tell you why this happens but if you need more info, I would be glad to help

shdunning commented 8 years ago

It would be helpful if I could try to reproduce the problem using your source, or some small app derived from your source that still exhibits the issue. I tried, unsuccessfully, to repro yesterday using a new app built on the SDK.

--Shaun

On Oct 11, 2016, at 4:54 AM, Christophe668 notifications@github.com wrote:

I tried on a Nexus 6P and a Nexus 5X on Android 7.0 but it doesn't seems to fix the problem unfortunately. I can't tell you why this happens but if you need more info, I would be glad to help

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

Christophe668 commented 8 years ago

Hi @shdunning,

You can find a simple example demo here which fails on a physical Nexus 6P 7.0 : https://github.com/Christophe668/UserVoiceNSupportBugdemo

The scenario is exactly the same. I'm just doing an initialisation and launching the contact form directly after a click on the fab. The project was generated wuth the Android Studio new project's wizard. Nothing else was added or modified.

I hope this will help you.

shdunning commented 8 years ago

@Christophe668 unfortunately, I was able to run your app successfully on a Nexus 5X running 7.0 (I don't have access to a 6P). Let's rule out some simple dev env configuration settings. Here's what I've been building with (nothing too out of the ordinary):

Also, what subdomain are you using to initialize your config in the main activity? Can you try demo.uservoice.com and see if you still experience the issue? For example,

        Config config = new Config("demo.uservoice.com");
Christophe668 commented 8 years ago

I was able to test it on a Nexus 5x 7.0 today and it works well this time. It's probably not a configuration issue then. You can see the bug on this video : http://sendvid.com/3nutrgh1 The domain we are using is seristapp.uservoice.com.

shdunning commented 8 years ago

I was just able to successfully run your app on a Nexus6P moments ago:

https://cl.ly/311I2N3u1w3e

One potentially useful piece of information is the phone was recently updated last night. Can you ensure that your device and build env are completely updated?

shdunning commented 8 years ago

@Christophe668 we also have another independent confirmation that UserVoice SDK is working now with latest Nexus 6P updates.

Christophe668 commented 8 years ago

Thanks for the info, unfortunately I won't be able check further why this is happening on my device in the coming days. If I have more time in the future, I will inform you if I find somthing.