Skip to content
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.

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

Revision suggestion: DRY #59

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jasonkhanlar
Copy link

@jasonkhanlar jasonkhanlar commented Feb 11, 2017

Revision suggestion: DRY, prepare static HTML to be generated dynamically via wpoa_generate_login_section() function whilst also accounting for potential unique capabilities for each OAuth service provider (showing content before and/or after the main options).

I would like to add Discord OAuth capabilities to this WordPress plugin, but first I thought it may be useful or helpful to provide this revision.

Thoughts?

…ally via wpoa_generate_login_section() function whilst also accounting for potential unique capabilities for each OAuth service provider.
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.

1 participant