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.

Sign up for GitHub

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

Jump to bottom

ycb only replaces one $$param$$ per value #13

Closed
isao opened this issue May 3, 2013 · 1 comment
Closed

ycb only replaces one $$param$$ per value #13

isao opened this issue May 3, 2013 · 1 comment

Comments

@isao
Copy link
Contributor

isao commented May 3, 2013

@proverma

Currently if a string has two ycb params, like
"page" : "$$baseUrl$$/$$path$$"

Only first param gets replaced by ycb, and second param gets left unreplaced.
@nagc
Copy link

nagc commented May 3, 2013

Thank you Isao for looking into it. We highly appreciate your quick help on this.

isao pushed a commit to isao/ycb that referenced this issue May 7, 2013
fix issue yahoo#13 replace multiple tokens per value, if possible
3febd6d 
if a string value contains multiple $$tokens$$, replace them
all if they resolve to a value. todo: needs refactoring
isao pushed a commit that referenced this issue May 8, 2013
Merge pull request #15 from isao/multisub
e49419b 
fix issue #13 ycb only replaces one $$param$$ per value
@isao isao closed this as completed May 8, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@isao @nagc