Skip to content

Conversation

@Thaina
Copy link

@Thaina Thaina commented Feb 1, 2025

  • Move Sample to Package/Sample~
  • Using raw java file to build instead of rely on gradle build
  • Patching androidmanifest with AssetImporter
  • Move MenuItem out to "Google/"
  • Use PlayerProfile directly

@Davidnovarro
Copy link

We need this :)

@miladEsk
Copy link

miladEsk commented Apr 8, 2025

Hello, If I use this branch instead of master could fix this issue I have:
Getting this ERROR: Returning an error code #3348

@RaphaelNSG
Copy link

Hello @Thaina ,

With new version of Unity the GPGSUtil.cs it's not finding the the plugin folder when using UPM. Can you already fix this on this Pull Request?
Issue: 250742

@Thaina
Copy link
Author

Thaina commented May 7, 2025

@RaphaelNSG I use my package in many version include 6000 and never see that problem

@RaphaelNSG
Copy link

@Thaina With me and with the people on this (#3163) issue is happening. Did you tried with Unity 2022?
My Unity is 2022.3.58f1 and my OS is MacOS 14

@Thaina
Copy link
Author

Thaina commented May 7, 2025

Not sure about osx, have you tried with 6000 yet?

@RaphaelNSG
Copy link

Actually o notice that in you branch the GPGSUilt.cs its already updated to search in the cache package:

#if UNITY_2018_4_OR_NEWER
                        // search for remote UPM installation
                        Path.Join("Library","PackageCache"),
                        "Packages",
#endif

I tried with the master branch. Sorry

@ErnSur
Copy link

ErnSur commented Jul 14, 2025

You could update the changelog to mention all changes like changing the menu item paths. It would also make the review easier

GameInfo.WebClientId,
forceRefreshToken,
javaScopesList))
using (var task = client.Call<AndroidJavaObject>("requestServerSideAccess", WebClientId, forceRefreshToken))

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Thaina javaScopesList is missing
Raises error
System.Exception: No such proxy method: GooglePlayGames.Android.AndroidTaskUtils+TaskOnSuccessProxy1[UnityEngine.AndroidJavaObject].onSuccess(System.String)`

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you try fork this and change task.AddOnSuccessListener<AndroidJavaObject> to task.AddOnSuccessListener<string> instead?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Most likely it helps, but why?
Above there is a separate RequestServerSideAccess method without scopes support. And this method is specifically for a request with scopes specified. Returning javaScopesList to the list as an argument fixes the problem

- using (var task = client.Call<AndroidJavaObject>("requestServerSideAccess", WebClientId, forceRefreshToken))
+ using (var task = client.Call<AndroidJavaObject>("requestServerSideAccess", WebClientId, forceRefreshToken, javaScopesList))

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mizunokazumi Sorry I misread the problem

Request serverside access was just added recently. I don't have time to update yet

https://github.com/playgameservices/play-games-plugin-for-unity/releases/tag/v2.1.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants