-
Notifications
You must be signed in to change notification settings - Fork 29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[ADDED] Connection type "in process" #221
Conversation
Added `ConnectionTypeInProcess` to the `UserPermissionLimits.AllowedConnectionTypes` list of possible types. Signed-off-by: Ivan Kozlovic <[email protected]>
@aricart As usual, doing this PR and once merged, would you mind doing a release? I have a server PR that will need to be updated to point to proper commit. |
Pull Request Test Coverage Report for Build 8995033103Details
💛 - Coveralls |
@aricart I am waiting on some feedback from the user on the NATS Server issue, so please hold off the review until I hear back. I will post here when ready for review. Switching to Draft to make sure there is no merge by mistake. |
@aricart Ok, I heard from the user and that will work. So you can review now. Thanks! |
Doing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@aricart Let me know if you can do a release. If not, I will reference the commit from main first and update when the release will be available. |
@kozlovic let me cut a release |
@kozlovic done!! |
@aricart Thanks! |
Added
ConnectionTypeInProcess
to theUserPermissionLimits.AllowedConnectionTypes
list of possible types.Signed-off-by: Ivan Kozlovic [email protected]