-
Notifications
You must be signed in to change notification settings - Fork 17
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
Added square example (#41) #73
Conversation
src/square.ts
Outdated
"Square create customer", | ||
async () => { | ||
// Use the Square SDK to send a message to the seller | ||
const customersApi = squareClient.customersApi; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't quite get this extra step, please see the next comment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok.. I did split because of makes more readable.
src/square.ts
Outdated
// After create a customer you can see it in the Square dashboard | ||
// at https://squareup.com/dashboard/customers/directory/all | ||
// See more https://developer.squareup.com/reference/square/customers-api/create-customer | ||
await customersApi.createCustomer(payload); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't we just do this?
await customersApi.createCustomer(payload); | |
await squareClient.customersApi.createCustomer(payload); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your submission! Left some comments. Also, if you could format this properly please. There are twice the amount of spaces (I think).
Thanks. Review again.. |
Looks good! |
@biplobsd this is great, could you fix conflicts pls and ill merge |
@D-K-P Fixed! |
Closed: #41
Summary
Create contract in the
customersApi
apiScreen record
Square-example.webm