Skip to content

Fixes #1762 - GeoStore API compatibility#1814

Merged
Gnafu merged 1 commit into
geosolutions-it:masterfrom
Gnafu:permission_loader_fix
May 12, 2017
Merged

Fixes #1762 - GeoStore API compatibility#1814
Gnafu merged 1 commit into
geosolutions-it:masterfrom
Gnafu:permission_loader_fix

Conversation

@Gnafu

@Gnafu Gnafu commented May 11, 2017

Copy link
Copy Markdown
Contributor

Il the user has only one group, GeoStore does not emit an Array, but a single Object

Il the user has only one group, GeoStore does not emit an Array, but a single Object
@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.02%) to 79.292% when pulling cdeceda on Gnafu:permission_loader_fix into a99bac6 on geosolutions-it:master.

@offtherailz offtherailz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'd like to invert the logic of that if (using _.get) , but anyway it seems ok

@Gnafu Gnafu merged commit c4112e5 into geosolutions-it:master May 12, 2017
@Gnafu Gnafu deleted the permission_loader_fix branch June 1, 2017 13:38
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.

3 participants