Skip to content
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

DEVENGAGE-342: Fixed string interpolation bug in tutorial #61

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

carnellj
Copy link
Contributor

Fixed based on bug from Jerome.

Copy link
Collaborator

@timsmithgenesys timsmithgenesys left a comment

Choose a reason for hiding this comment

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

Need to escape double curlys; syntax is reserved for templating.

@carnellj carnellj closed this Feb 17, 2021
@carnellj carnellj reopened this Feb 17, 2021
@carnellj
Copy link
Contributor Author

Need to escape double curlys; syntax is reserved for templating.

So do I need to do anything here. I think Jerome mistook these to need to be parameterized with double {{}}. Let me know and I will close the pull request.

@timsmithgenesys
Copy link
Collaborator

Need to escape double curlys; syntax is reserved for templating.

So do I need to do anything here. I think Jerome mistook these to need to be parameterized with double {{}}. Let me know and I will close the pull request.

@carnellj If the double curly syntax is the correct way to use that URL, it needs to be updated to escape the double curlys. If this was a misunderstanding about dev center syntax, it can be closed. I'm not familiar enough with the subject matter to know what the intention of this change is.

@themooks
Copy link

Yes, the double curly syntax is the correct way.
When defining a Client App integration, you need {{}} syntax for the URL interpolation to trigger/work in Genesys Desktop.

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.

4 participants