Skip to content
This repository has been archived by the owner on Feb 1, 2021. It is now read-only.

FOGL-2773: Allow service name with double quotes escaped in FogLAMP C… #1541

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

pintomax
Copy link
Contributor

…++ classes

FOGL-2773: Allow service name with double quotes escaped in FogLAMP C++ classes

…++ classes

FOGL-2773: Allow service name with double quotes escaped in FogLAMP C++ classes
@pintomax pintomax added the DO NOT MERGE YET This PR is subject to further development and testing label Apr 30, 2019
@pintomax pintomax marked this pull request as ready for review April 30, 2019 15:30
@pintomax pintomax removed the DO NOT MERGE YET This PR is subject to further development and testing label Apr 30, 2019
Unit tests updated
Copy link
Contributor

@Singhal-Vaibhav Singhal-Vaibhav left a comment

Choose a reason for hiding this comment

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

Please refer JIRA for change request

@pintomax pintomax requested a review from Singhal-Vaibhav May 29, 2019 13:56
Copy link
Contributor

@MarkRiddoch MarkRiddoch left a comment

Choose a reason for hiding this comment

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

LGTM - having " in names is a slightly questionable thing anyway

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants