-
Notifications
You must be signed in to change notification settings - Fork 34
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
New custom client, error in pulling previous data streams. #324
Comments
Good job using markdown! I just reformatted it to make it easier to read. Since we are using GitHub, you can use GitHub-flavored markdown, which allows you to use ``` for multi-line text |
wrt the actual error, I am pretty sure that I know what it is, but I checked and I believe it is already covered in the documents. Just so I can confirm this, can you please confirm that you have done the steps from https://github.com/e-mission/e-mission-docs/blob/master/docs/e-mission-phone/create_a_new_custom_client.md You can do this directly in markdown [x] copy over config.xml |
Note also that at this moment, you cannot push the built app directly to the play store because of Since there are multiple of you who want to deploy to the stores directly, I will fix that. However, before I do, I have pending changes to fix the "location tracking toggle" issue that I would like help testing. And of course, I would like help testing the dynamic permissions as well. So, in addition to resolving your build issues, can you also:
|
Please keep me updated on the status of the testing |
I have installed. I am experimenting will let you know tomorrow. |
1.copy over config.xml 2.change the package in the id 3.reset the android-versionCode, ios-CFBundleVersion and version
4 create a new appid in apple (https://developer.apple.com) with the new id specified in step 2 Not supporting iOS
Not supporting iOS
Don't want right now 7.copy over package.json and make the same changes as config.xml One thing I have noticed is that package.json takes preference. When I try to prepare cordova making changes in both files (config.xml & package.json) package.json tends to overwrites config.xml and the changes are not reflected in apk. the package.json will always be created to mirror the config.xml whenever you run cordova prepare, if it does not exist. If a package.json does exist, it will take preference (but only for the things defined in it, like plugins and platforms, for other config options config.xml will still be used). |
ok... but you are making the same changes to package.json and config.xml, so it doesn't really matter which takes precedence. What about the other steps? There are more than 7... |
|
@deepalics0044 are you waiting for a response from me? If so, can you specify what your question is? |
Do the instructions say anything about I am not going to respond to any other comments until I see a complete listing of all the steps in the instructions and whether you have executed them. |
The above step changed the config.xml with new version and versionCode and package.json with previous . I made the changes in package.json again which matches config The steps involves copy over config.xml,change the package in the id,give app a new name i.e Logtrip
|
What I get in server terminal while processing data
|
|
Not sure what you mean by this. Any local folder you have is a clone of the repo, isn't it? |
Yes
…On Tue, Mar 5, 2019, 8:06 PM shankari ***@***.***> wrote:
If you want to use google auth, create google auth credentials correctly
for the new package name and include them into
www/json/connectionConfig.json
Note:- that i did'nt clone the repo instead worked on the copy of the
local folder
Not sure what you mean by this. Any local folder you have is a clone of
the repo, isn't it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#324 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AnBeHeqNlc6vxz-YsdFtGRoTmvFZeXrnks5vToDzgaJpZM4bWMXX>
.
|
So what does that statement mean "I didn't clone the repo instead worked on the copy of the local folder" |
I didn't actually clone the e-mission-phone repository. I already had it,
so I made a local folder where I copied all the files, so that I don't have
to make changes in configuration files. I just wanted to make sure that's
not causing the problem.
…On Tue, Mar 5, 2019, 8:19 PM shankari ***@***.***> wrote:
So what does that statement mean "I didn't clone the repo instead worked
on the copy of the local folder"
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#324 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AnBeHaym4KZtkdCeE_hEhXGk8Vl2iC8Oks5vToQCgaJpZM4bWMXX>
.
|
That is almost certainly the problem. Note the last instruction
You can't just use the old configuration if you are making a new app with a new package name |
After changing the Google auth clientID with the new one in
connectionConfig.json. I am getting the same
Keyerror. I changed the auth on server end but then also I get the same key
error.
…On Tue, Mar 5, 2019, 8:29 PM shankari ***@***.***> wrote:
so that I don't have to make changes in configuration files.
That is almost certainly the problem. Note the last instruction
If you want to use google auth, create google auth credentials correctly
for the new package name and include them into
www/json/connectionConfig.json
You can't just use the old configuration if you are making a new app with
a new package name
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#324 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AnBeHX4oy99XFEprjqw085yBxraEkOIAks5vToZAgaJpZM4bWMXX>
.
|
The This means that the user authentication is not working on the phone. You need to ensure that you followed all the phone instructions. Maybe check the phone logs for more clues? What are the steps that you took to replace the In general, just telling me that something "doesn't work" does not help because I don't know what steps you did or did not do. If you made changes, please list out the exact steps that you did. Other groups (@jf87, @PatGendre) have successfully built their own apk recently, so there is nothing fundamentally broken in the build process. |
@deepalics0044 any updates on this? |
I got the sha1 and package name from android build project and created a new client ID then replaced the old with new in connectionConfig.json and rebuild the cordova . But again the same error |
Look at the phone logs and see what error you get during token generation. Finally, your steps are still not clear - which clientID did you use? There are different kinds of client IDs that can be created. Details matter a lot for google auth which is super picky. |
The one for Android app. |
Aha! Check the documentation for google auth :)
|
* Clarify documentation around google auth This clarifies #324 * Add the screenshot to the correct folder * Fixed link to image file
On re-reading the authentication documentation, I realized that the google auth section could use clarification. So I have now added a screenshot that hopefully makes it clearer. |
Totally fine to upload pictures here. not much else you can do with GUI. the markdown was really for CLI So I don't ever get questions on verification, so let me upload the screenshots of how to enable this. |
Uploaded screenshots as part of #335 |
nevertheless I am going to repeat the whole process again. |
Since you are testing this with the emulator anyway, did you try running in the debugger and putting breakpoints as suggested in #324 (comment) |
Shouldn't need to repeat the whole process, rebuilding cordova should be good enough. |
@deepalics0044 any updates on running in the debugger with breakpoints? |
Nope.
…On Fri, Mar 8, 2019, 9:09 PM shankari ***@***.***> wrote:
@deepalics0044 <https://github.com/deepalics0044> any updates on running
in the debugger with breakpoints?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#324 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AnBeHcy8ZnU2RMYMt2NUfoz6OX1oyhWbks5vUoQsgaJpZM4bWMXX>
.
|
@deepalics0044 can you please do so and update the results here before our meeting? |
|
Right, so clearly the error is and if you search for that error you find playgameservices/play-games-plugin-for-unity#1328 (comment) |
and also https://tinyapps.blogspot.com/2016/07/googleauthexception-invalidaudience.html It is definitely a configuration issue on your end. |
Okay I'll try to figure it.
…On Fri, Mar 8, 2019, 10:53 PM shankari ***@***.***> wrote:
and also
https://tinyapps.blogspot.com/2016/07/googleauthexception-invalidaudience.html
It is definitely a configuration issue on your end.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#324 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AnBeHe2EfeZQGxnSOV8Hb6P-yjcyBro9ks5vUpyGgaJpZM4bWMXX>
.
|
just join the call and let's see if I can help you figure it out. you also have to finish iOS and you don't have much time. |
I just came out of the department it closes at 11pm. Can you please set up
a meeting tomorrow morning
…On Fri, Mar 8, 2019, 10:56 PM shankari ***@***.***> wrote:
just join the call and let's see if I can help you figure it out. you also
have to finish iOS and you don't have much time.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#324 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AnBeHcFzRmzjTJ1tss45zL057VMSff5oks5vUp1WgaJpZM4bWMXX>
.
|
Error on server terminal
|
The error is here
We are trying to load the key file and we are getting a decode error. The key file is not valid json. Please check the format. As documented https://github.com/e-mission/e-mission-docs/blob/master/docs/e-mission-both/configuring_authentication.md#google_authgoogle-authutil |
|
For the record, the issue was that @deepalics0044 had only the ios key and the webserver key in the project, not an android key. For anybody else who finds this issue and is trying to debug their own environment, the Configuration section on google authentication
and that is also indicated in the associated screenshot. Those instructions are real. Please make sure that you have all three of the keys. |
@deepalics0044 if this has resolved the issue, please close it |
* Clarify documentation around google auth This clarifies #324 * Add the screenshot to the correct folder * Fixed link to image file
I am working on the android bulid e-mission project in android-studio. After assigning different name, logo and version I can see the changes in the emulator. The only problem is I am not able to retrieve previous trip data. Whenever I try to process data, this is what I see in server log.
I just want to make sure I get all data before launching it on play store.
The text was updated successfully, but these errors were encountered: