Skip to content

Version 0.5.2#284

Merged
samtstern merged 3 commits intofirebase:masterfrom
samtstern:master
Sep 6, 2016
Merged

Version 0.5.2#284
samtstern merged 3 commits intofirebase:masterfrom
samtstern:master

Conversation

@samtstern
Copy link
Contributor

Changes:

@puf please review.

  * Scopes for Google (firebase#281)
  * Gradle build fix (firebase#272)
  * README colorization for code snippets

Change-Id: Ie35a987a027307f3eb092a241b6ca1ce1f21095d
Change-Id: Idb54fa4936e6c8af5db8f21e9d1c2ae0d9996729
auth/README.md Outdated
#### Google
By default, FirebaseUI requests the `email` and `profile` scopes when using Google Sign In. If you
would like to request additional scopes from the user, add a string array resource to your
application like this:
Copy link
Contributor

Choose a reason for hiding this comment

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

Where do I add this? A new element in an existing file? Or is it a new file? If so, does the name of the file matter? Or is it only about the array's name?

@puf
Copy link
Contributor

puf commented Sep 3, 2016

A few comments. Aside from that lgtm.

Change-Id: Ida7b0025654e25158f466074ce2d3a0674b577d6
@samtstern
Copy link
Contributor Author

@puf clarified XML setup

@samtstern samtstern merged commit 4ea4fb5 into firebase:master Sep 6, 2016
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.

2 participants