Skip to content
This repository has been archived by the owner on Apr 23, 2021. It is now read-only.

Updated "How to configure sandbox environment" #72

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Updated "How to configure sandbox environment" #72

wants to merge 1 commit into from

Conversation

frankely
Copy link

I renamed the client variable to defaultClient in the "How to configure sandbox environment" code sample to keep consistency with the name given to the SquareConnect.ApiClient.instance in the first "Getting Started" code sample. Because in my case, I copied and pasted the first example and then just copied the line that sets the basePath, and because the naming is different, my code was failing. Hopefully, this fix in the documentation will allow distracted copy-pasters to move along seamlessly.

I renamed the client variable to defaultClient in the "How to configure sandbox environment" code sample to keep consistency with the name given to the SquareConnect.ApiClient.instance in the first "Getting Started" code sample. Because in my case, I copied and pasted the first example and then just copied the line that sets the basePath, and because the naming is different, my code was failing. Hopefully, this fix in the documentation will allow distracted copy-pasters to move along seamlessly.
@ssung88
Copy link
Contributor

ssung88 commented Jan 2, 2020

The fix is merged here #80. Thanks for using our SDK and reporting!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants