Skip to content

Extra api calls to extend the go library#12

Open
pjmuller wants to merge 1 commit into
andreas:masterfrom
pjmuller:extra_api_wrappers
Open

Extra api calls to extend the go library#12
pjmuller wants to merge 1 commit into
andreas:masterfrom
pjmuller:extra_api_wrappers

Conversation

@pjmuller

@pjmuller pjmuller commented Jul 3, 2016

Copy link
Copy Markdown
Contributor

Some extra methods.

This is my first go code, I followed your lead. But if there are things that seem off.

Just let me know and I'll correct

Comment thread app.go
}

// https://developers.podio.com/doc/applications/get-space-app-dependencies-45779
func (client *Client) GetSpaceDependencies(spaceId int64) (response *interface{}, err error) {

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

I think you should define the return type -- I'm not sure how this is supposed to work?

@andreas

andreas commented Jul 4, 2016

Copy link
Copy Markdown
Owner

A couple of comments, but otherwise looks good 😄

@pjmuller

pjmuller commented Jul 5, 2016

Copy link
Copy Markdown
Contributor Author

Ok clear comments and I think I know what to change. Will commit later this week probably

@pjmuller

Copy link
Copy Markdown
Contributor Author

Hi Andreas, got a bit (or even a lot of delay) on this one. Haven't forgotten however ;).

Will make the tiny changes in a couple of weeks normally (if all goes according to plan)

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