Skip to content

Conversation

@SUPERCILEX
Copy link
Collaborator

More work on #1196.

For some reason, I just can't get login to work with Play Games. I'm always getting error code 12501 and logs like this: GamesAuthenticator: Must have a game ID to sign in!. Normal Google sign-in works just fine and I've tried hardcoding the client id, messing with SHA1 certificates, everything! Could you perhaps try it with your setup to see if it's just me? 🤔

Signed-off-by: Alex Saveau <saveau.alexandre@gmail.com>
@SUPERCILEX SUPERCILEX requested a review from samtstern as a code owner March 25, 2018 19:14
@samtstern samtstern added this to the 3.4.0 milestone Mar 26, 2018
Signed-off-by: Alex Saveau <saveau.alexandre@gmail.com>
# Conflicts:
#	auth/src/main/java/com/firebase/ui/auth/util/data/ProviderUtils.java
Signed-off-by: Alex Saveau <saveau.alexandre@gmail.com>
@SUPERCILEX SUPERCILEX changed the base branch from version-3.3.0-dev to master March 28, 2018 21:54
@samtstern samtstern changed the base branch from master to version-3.3.1-dev March 29, 2018 15:29
@samtstern samtstern changed the base branch from version-3.3.1-dev to version-3.4.0-dev April 13, 2018 19:48
@SUPERCILEX
Copy link
Collaborator Author

@samtstern Do you want to take a look at this one? I'm gonna be honest, I couldn't get it to work, see #1211 (comment).

@samtstern
Copy link
Contributor

@SUPERCILEX I'll see if I can get games sign in going.

@samtstern
Copy link
Contributor

@SUPERCILEX update: I did look at this but I am also having a hell of a time actually getting set up with Play Games ... and I used to work on that product!

@LayoutRes int buttonLayout;
switch (idpConfig.getProviderId()) {
case GoogleAuthProvider.PROVIDER_ID:
case PlayGamesAuthProvider.PROVIDER_ID:
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we want to have a green "Sign in with Play games" button. I'll see if I can get us the assets.

@samtstern
Copy link
Contributor

Got some assets for you, let me know if we need different ones.

image

play-games-icons.zip

@SUPERCILEX
Copy link
Collaborator Author

Sweet, thanks for the assets! And yeah, let me know if you ever find someone who's able to get Play Games sign-in to work 😂

Signed-off-by: Alex Saveau <saveau.alexandre@gmail.com>
…both

Signed-off-by: Alex Saveau <saveau.alexandre@gmail.com>
Signed-off-by: Alex Saveau <saveau.alexandre@gmail.com>
@SUPERCILEX
Copy link
Collaborator Author

@samtstern We're also going to need translations, I'll let you get that started. 😊

@samtstern samtstern changed the base branch from version-3.4.0-dev to version-4.0.0-dev May 4, 2018 17:25
…ames

# Conflicts:
#	library/quality/quality.gradle
@samtstern samtstern modified the milestones: 4.0.0, 4.1.0 May 15, 2018
@samtstern
Copy link
Contributor

@SUPERCILEX offended if I kick this and Github to 4.1.0? We've got a lot in flight right now and I don't think we'll get this working in time, and the TOS/PP changes have a pretty hard deadline with GDPR and whatnot.

@SUPERCILEX
Copy link
Collaborator Author

Not at all! That makes sense to me. 👍

@SUPERCILEX SUPERCILEX changed the base branch from version-4.0.0-dev to version-4.1.0-dev May 29, 2018 00:00
@samtstern samtstern modified the milestones: 4.1.0, 4.2.0 Jun 25, 2018
@samtstern
Copy link
Contributor

I still can't get PGS sign in to work ... so bumping to 4.2.0. Sorry @SUPERCILEX I know you worked hard on this!

@SUPERCILEX
Copy link
Collaborator Author

Haha, well if no one can get this working, maybe you could find someone who worked on it and show us how it's done. 😁

@pushbit
Copy link

pushbit commented Aug 24, 2018

Any luck getting help from an insider? Years ago I was trying to implement GCM device group messaging and after following the documentation to the letter it just wouldn't work. Many months later a Googler replied on Stack Overflow (or similar) and revealed that the request needs an undocumented parameter, and then it finally worked. I get the feeling this is a similar situation.

@samtstern
Copy link
Contributor

Still having no luck here. My attempts fail with an error in the developer console, and a lot of times in those cases being an insider is the problem not the solution.

@samtstern samtstern removed this from the 4.2.0 milestone Sep 12, 2018
@samtstern
Copy link
Contributor

Sigh. Booting this from the milestone again.

@samtstern
Copy link
Contributor

@SUPERCILEX going to close this old PR so that there's not confusion like there was on #309

@samtstern samtstern closed this Jan 11, 2022
@SUPERCILEX
Copy link
Collaborator Author

Sounds good!

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.

3 participants