-
Notifications
You must be signed in to change notification settings - Fork 36
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
return str.replace(htmlEntitiesRegex, (match, dec : str.replace is not a function #54
Comments
HI @DNeelanshi, Great observation, I see what you mean. Please feel free to submit a PR with this change if you'd like to contribute! For internal tracking, DTBTSDK-2818 |
Thank you @oscarleonnogales for your response , I will try to contribute by forking to fix this issue . |
please review PR #56 |
Thank you for starting the PR @DNeelanshi, I went ahead and did a few other changes along with your code and added you as a co-author for the final commit. Closing this issue now as the PR has been merged and will be out in the next release. |
Null check is needed on str as its throwing error that str.replace is not a function
The text was updated successfully, but these errors were encountered: