8000 Add Twitch OAuth2 Support by donofriov · Pull Request #84 · appwrite/appwrite · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add Twitch OAuth2 Support #84

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 66 commits into from
Closed

Add Twitch OAuth2 Support #84

wants to merge 66 commits into from

Conversation

donofriov
Copy link
  1. Autogenerated composer.lock and package-lock.json files via install commands.
  2. Added Twitch provider and alphabetized list.
  3. Added Twitch OAuth2 Support based on issue [HELP-WANTED] Adding Support for More OAuth Providers #5.

cho0o0 and others added 30 commits September 28, 2019 14:55
add ZH locale 10000
d19a1dd
Added hungarian translations
09d4785
Improve IT translations
a069aab
Fix PR conflicts
03537f2
@panz3r
Merge branch 'master' into improve-IT-translations
cd9b496
@lex111
Replace Github with GitHub
e55019e
@lex111
Merge branch 'master' into gh-fix
2bd6fac
@MasterKN48
Create cz.email.auth.confirm.tpl
4e19fb8
@MasterKN48
Create cz.email.auth.invitation.tpl
e93baf3
@MasterKN48
Create cz.email.auth.recovery.tpl
ba0cc7d
@MasterKN48
Create cz.php
aec55d8
@MasterKN48
Create cz.countries.php
7f97b2f
@MasterKN48
Create cz.continents.php
a58b4d9
@christyjacob4
fix: removed instagram oAuth due to technical difficulties
5167ce8
@christyjacob4
Merge remote-tracking branch 'upstream/master'
1d672bf
revert docker compose
2ba4aa5
@daniloff200
add RU locale
3d93be5
Add Twitch OAuth2 Support
3eb4c3d
Add Vietnamese for email template
400c332
Merge branch 'master' into master
31d2053
@anirudh-kac
Fix grammar in README
79916eb
@christyjacob4
feat: added Microsoft OAuth Support
eb47cf8
@christyjacob4
chore: updated Microsoft Image
f89b7c0
@christyjacob4
chore: lint fix
119f5de
@christyjacob4
feat: started Reddit OAuth
2e42873
@dimshik100
fixed typo
783601c 
Fixed typo `symol` -> `symbol` :)
My samll contribution to the environment.
@eldadfux
Merge pull request appwrite#96 from dimshik100/master
817cfed 
fixed typo
@parshnt
Fixed Grammer and Typo
ecc22d1
@eldadfux
Updated credits
48ef680
@Ratna04priya
editing grammar
e27544c 
local host <-> localhost
eldadfux
eldadfux requested changes Oct 2, 2019
View reviewed changes
src/Auth/OAuth/Twitch.php
// Reference Material
// https://dev.twitch.tv/docs/authentication/getting-tokens-oauth/

class Twitch extends OAuth
Copy link
Member

Choose a reason for hiding this comment

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

Missing some implementation of some mission critical abstract methods.
Please check what @christyjacob4 wrote about this issue here:
#5 (comment)

@eldadfux
Copy link
Member
eldadfux commented Oct 2, 2019

You can find more info here:
#93 (comment)

@donofriov donofriov closed this Oct 3, 2019
@donofriov donofriov deleted the feature/twitch-oauth-support branch October 3, 2019 12:09
@donofriov donofriov mentioned this pull request Oct 3, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eldadfux eldadfux eldadfux requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
0