-
-
Notifications
You must be signed in to change notification settings - Fork 14
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
Update AndroidManifest.xml #35
base: master
Are you sure you want to change the base?
Conversation
Removed the preceding slash that was preventing GPSP to work properly.
with without this symbol it should be path to the string eg |
Hmm, this value has changed a few times. Most recent change: https://github.com/defold/extension-gpgs/blame/master/gpgs/manifests/android/AndroidManifest.xml#L6 @SkaterDad GPGS is working as it is now right? @AGulev good suggestion btw! |
keeping this value in "@string/app_id" is the safest way I think, but isn't so convenient. |
otherwise parser is looking for |
@britzl the current version works for me. I haven't tried building for a month or so, but the published game is fine. |
So there are conflicting suggestions here. The PR suggests dropping the space while one user confirms that it works as it is. I do not know what is correct. One option may be to change how GPGPS is initialized and let the user do it himself or do it from C (and read the config value from game.project) |
Removed the preceding slash that was preventing GPSP to work properly.