Skip to content

Conversation

@hnanchahal
Copy link
Contributor

@hnanchahal hnanchahal commented Nov 20, 2020

…lease.

Closes #1035

Copy link
Contributor

@amanda11 amanda11 left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution, this is great. Just a couple of minor comments I had.

Copy link
Member

@arm4b arm4b left a comment

Choose a reason for hiding this comment

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

Things look better, Thanks! 👍

Just left a couple more minor comments to address and we're good to merge it.

Copy link
Member

@arm4b arm4b left a comment

Choose a reason for hiding this comment

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

Looks good to me! 👍
Thanks for the documentation update for LDAP!

@arm4b arm4b requested a review from amanda11 December 3, 2020 14:29
@amanda11 amanda11 added this to the 3.4.0 milestone Dec 3, 2020
Copy link
Contributor

@amanda11 amanda11 left a comment

Choose a reason for hiding this comment

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

LGTM

@arm4b arm4b merged commit ed2525d into StackStorm:master Dec 3, 2020
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.

LDAP update following the st2 core integration from Enterprise

3 participants