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

Allow a url to be passed in to load after login #41

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

Conversation

supersoju
Copy link

Addresses #40

Added support for an optional --redirect-to= param that will send the user to the specified WordPress admin page after following the link.

one-time-login.php Outdated Show resolved Hide resolved
one-time-login.php Outdated Show resolved Hide resolved
@danielbachhuber
Copy link
Owner

@supersoju Are you up for adding some tests to this PR? We'd ideally add some around one_time_login_handle_token() and the behaviors with and without a redirect value.

@danielbachhuber
Copy link
Owner

Are you up for adding some tests to this PR?

@supersoju Please merge master to apply #43. Once you've done so, it'd be great to have tests for the _valid_redirect_to and _invalid_redirect_to scenarios.

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