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

From connect-api-specification: Merge pull request #53 from square/tr… #31

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

Conversation

tristansokol
Copy link
Collaborator

…istans/fix-payment-model

update data type for v1payments>creator_id, bump versions, remove out…

…istans/fix-payment-model

update data type for v1payments>creator_id, bump versions, remove out…
@tristansokol tristansokol requested a review from bunnyc1986 June 20, 2017 00:30
README.md Outdated
@@ -73,8 +73,8 @@ namespace Example
// Retrieving your location IDs
public void RetrieveLocations()
{
LocationsApi _locationsApi = new LocationsApi();
var response = _locationsApi.ListLocations();
LocationApi _locationApi = new LocationApi();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think LocationsApi and TransactionsAPI are correct.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ahh, a bug in the templates. Fixed in square/connect-api-specification#57 and this repo addressed in 898e862

@tristansokol tristansokol mentioned this pull request Jun 22, 2017
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