Skip to content

Fix client SQL manager missing client_name#303

Merged
aeneasr merged 1 commit intoory:masterfrom
johnwu96822:sql_client_name_fix
Nov 5, 2016
Merged

Fix client SQL manager missing client_name#303
aeneasr merged 1 commit intoory:masterfrom
johnwu96822:sql_client_name_fix

Conversation

@johnwu96822
Copy link
Copy Markdown
Contributor

Found the bug when I couldn't set/update the client name of a client.

Signed-off-by: John Wu johnwu96822@gmail.com

@aeneasr
Copy link
Copy Markdown
Member

aeneasr commented Nov 4, 2016

Thanks! Could you add a test case for it too? I think it should be enough to add the additional fields here and here.

Signed-off-by: John Wu <johnwu96822@gmail.com>
@aeneasr
Copy link
Copy Markdown
Member

aeneasr commented Nov 5, 2016

Thanks!

@aeneasr aeneasr merged commit b861e25 into ory:master Nov 5, 2016
@johnwu96822 johnwu96822 deleted the sql_client_name_fix branch November 7, 2016 17:17
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