Skip to content

client: fix client.GetClients() for multiple clients#151

Merged
aeneasr merged 1 commit intoory:masterfrom
boyvinall:boyvinall/fix-clients
Jul 21, 2016
Merged

client: fix client.GetClients() for multiple clients#151
aeneasr merged 1 commit intoory:masterfrom
boyvinall:boyvinall/fix-clients

Conversation

@boyvinall
Copy link
Copy Markdown
Contributor

fix for #150

Signed-off-by: Matt Vinall boyvinall@gmail.com

fix for ory#150

Signed-off-by: Matt Vinall <boyvinall@gmail.com>
@aeneasr
Copy link
Copy Markdown
Member

aeneasr commented Jul 21, 2016

Could you please add a test case as discussed on gitter? It would be awesome to automate the check :)

@aeneasr
Copy link
Copy Markdown
Member

aeneasr commented Jul 21, 2016

nvm, I'll create one myself. Thanks for this!

@aeneasr aeneasr merged commit 93dc837 into ory:master Jul 21, 2016
@boyvinall
Copy link
Copy Markdown
Contributor Author

sorry, just catching up again, thanks. If I get time then I'll add one in a bit..

@aeneasr
Copy link
Copy Markdown
Member

aeneasr commented Jul 21, 2016

already on it

@aeneasr
Copy link
Copy Markdown
Member

aeneasr commented Jul 21, 2016

see #152

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