-
Notifications
You must be signed in to change notification settings - Fork 32
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
feat: added url-sanitization function #110
feat: added url-sanitization function #110
Conversation
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.
Nice! Do we need to include changes in the dist
folder to belter PR's? 🤔
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.
Looks like this is using Braintree's sanitizeUrl
code, so why not just use that as a dependency?
https://github.com/braintree/sanitize-url
That was the original idea but since Belter itself is a repo for utils, Shane and I decided to make this util part of our repo as well. |
Good callout! looks like those were for some CI. Removed them. |
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.
LG2M
No description provided.