-
Notifications
You must be signed in to change notification settings - Fork 48
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
Make Encoder.encodeQueryElement public #41
Comments
Sometimes you need to use it where? A subclass of Encoder, or some other code which directly uses an Encoder instance? I'm trying to understand the use case first. |
I figured out that In order to use the above mentioned method in my codebase, I need to create a new subclass where I override the method as I thought it would be an improvement to make it public here, to avoid the hassle of creating subclasses of |
OK, sounds sensible. |
Just released 2.0.9 to Maven Central, should include this. |
Thanks @mikaelhg ! |
Sometimes you need to use
io.mikael.urlbuilder.util.Encoder.encodeQueryElement
and I extend theEncoder
to do that :) Would be great to make it public.The text was updated successfully, but these errors were encountered: