Skip to content

FB passport update#611

Merged
Martii merged 1 commit intoOpenUserJS:masterfrom
Martii:fbPassportDepUpdate
Mar 31, 2015
Merged

FB passport update#611
Martii merged 1 commit intoOpenUserJS:masterfrom
Martii:fbPassportDepUpdate

Conversation

@Martii
Copy link
Copy Markdown
Member

@Martii Martii commented Mar 31, 2015

  • passport-facebook - UNTESTED LOGIN BUT DEPLOYED and awaiting final test from @sizzlemctwizzle

NOTE:

  • passport-google-oauth requires Google+ API added to OUJS (google) account and can not be updated even without migration of scope. @sizzlemctwizzle will need to do this.

* *passport-facebook* - **UNTESTED LOGIN BUT DEPLOYED** and awaiting final test from @sizzlemctwizzle

**NOTE**:
* *passport-google-oauth* requires Google+ API added to OUJS account and can not be updated even without migration of scope. @sizzlemctwizzle will need to do this.
@Martii Martii added migration Use this to indicate that it may apply to an existing or announced migration. CODE Some other Code related issue and it should clearly describe what it is affecting in a comment. labels Mar 31, 2015
Martii added a commit that referenced this pull request Mar 31, 2015
@Martii Martii merged commit 4021df7 into OpenUserJS:master Mar 31, 2015
@Martii Martii deleted the fbPassportDepUpdate branch March 31, 2015 18:16
@Martii Martii added the needs testing Anyone can add this but it is primarily there for the Assignee indicating that Testers are wanted. label Mar 31, 2015
@Martii
Copy link
Copy Markdown
Member Author

Martii commented Mar 31, 2015

Passed test login from sizzle.

@Martii Martii removed the needs testing Anyone can add this but it is primarily there for the Assignee indicating that Testers are wanted. label Mar 31, 2015
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators May 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

CODE Some other Code related issue and it should clearly describe what it is affecting in a comment. migration Use this to indicate that it may apply to an existing or announced migration.

Development

Successfully merging this pull request may close these issues.

1 participant