search

3cixty / android-client-sample

Apache License 2.0
0 stars 0 forks source link

integrate with GoFlow sample app #3

Open pgraverdy opened 9 years ago

congkinh commented 9 years ago

Please check how to get this project started with Android Studio here

Please check how to include libraries in Android Studio here

pgraverdy commented 9 years ago

Added the jar and relevant code and resources. But still cannot resolve the classes from the jar. In the gradle config file, sudio complains about groovy.

I commited in a separate branch. Kinh, can you check what is wrong ?

congkinh commented 9 years ago

hi PG,

I reconfigured the build script with libs https://github.com/3cixty/android-client-sample/commit/e22024a6c164454c1bbad27fdce32f244cdebdc1. But, I realized that you only put code without importing classes. Could you complete your files? Thanks.

pgraverdy commented 9 years ago

done and comitted (took me some time as not fluent in android studio...)

I compiled and launched on my phone (but since only one activity, could not verify goflow? Can you add a second activity to the goflow test ? (I dont want to mess up the config)

congkinh commented 9 years ago

I will do that tomorrow as I have a bunch of work to fix right now

animeshpathak commented 9 years ago

So you want to update the code of MainActivity 's view so there is a button to launch the GoFlowMainActivity ?

pgraverdy commented 9 years ago

It was more defining another entry point (so that multiple icons on the android app list). I'll try to do that tonight.

animeshpathak commented 9 years ago

Wait, why is the assets file still having a uname/pwd combo? I thought we had moved past that no?

pgraverdy commented 9 years ago

the account code is not yet integrated. I did a quick copy of my test app inside the 3cixty sample that's why. It will take 1/2 day to complete.

pgraverdy commented 9 years ago

also to complete, it is already possible to not provide uname/pwd and set creds from the api, it's just that it is still possible to use the assets file and the test app was using this method. So the test app needs to be upgraded to fetch some uname/pwd from 3cixty using some gtoken.

animeshpathak commented 9 years ago

Ok.

BTW instead of having another entry point how about starting the goflow main activity using a new button on the current main activity?

Sent from my Android device with K-9 Mail. Please excuse my brevity.

animeshpathak commented 9 years ago

Indeed, that's what I had remembered, hence was surprised to see the uname/pwd combo in the file :)

Sent from my Android device with K-9 Mail. Please excuse my brevity.