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.

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

Revision suggestion: DRY #59

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

Revision suggestion: DRY #59

wants to merge 3 commits into from

Commits on Feb 11, 2017

  1. Revision suggestion: DRY, prepare static HTML to be generated dynamic… 

    …ally via wpoa_generate_login_section() function whilst also accounting for potential unique capabilities for each OAuth service provider.
    @jasonkhanlar
    jasonkhanlar committed Feb 11, 2017
    Configuration menu
    Copy the full SHA
    f053d39 View commit details
    Browse the repository at this point in the history

  2. WordPress checked() function echos and returns a value by default. Ch… 

    …anged to only return the value. See https://codex.wordpress.org/Function_Reference/checked
    @jasonkhanlar
    jasonkhanlar committed Feb 11, 2017
    Configuration menu
    Copy the full SHA
    4353944 View commit details
    Browse the repository at this point in the history

  3. get_option() function calls weren't returning values properly. Switch… 

    …ed ' to "
    @jasonkhanlar
    jasonkhanlar committed Feb 11, 2017
    Configuration menu
    Copy the full SHA
    4822c49 View commit details
    Browse the repository at this point in the history