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

Updated the PHP quick starts to remove the redirect and redirectLink methods #5

Merged
merged 2 commits into from May 13, 2019

Conversation

iammerus
Copy link
Contributor

Just updating the docs to ensure that people don't call methods that were removed after the docs were written.

Removed all mentions of the redirect() method and changed the redirectLink to redirectUrl following the standards laid out for the SDKs last year

Just updating the docs to ensure that people don't call methods that were removed after the docs were written.

Removed all mentions of the redirect() method and changed the redirectLink to redirectUrl following the standards laid out for the SDKs last year
@iammerus iammerus added the bug Something isn't working label May 13, 2019
@iammerus iammerus changed the title Jon Snow now knows something WIP: Jon Snow now knows something May 13, 2019
@iammerus
Copy link
Contributor Author

Following the comment by ferowise on the Paynow Forums, this update removes references to the non-existent methods redirect and redirectLink.

@spaghettiwews. How does one go about updating the slide thingie on the homepage of the docs site?

@iammerus
Copy link
Contributor Author

Never mind. Found it

Removed references to the redirect and redirectLink methods on the homepage
@iammerus iammerus changed the title WIP: Jon Snow now knows something Updated the PHP quick starts to remove the redirect and redirectLink methods May 13, 2019
@spaghettiwews spaghettiwews merged commit 28b100a into master May 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants