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

directory.orgunits.get params.orgUnitPath not encoded #620

Closed
wilau2 opened this issue Aug 22, 2016 · 2 comments
Closed

directory.orgunits.get params.orgUnitPath not encoded #620

wilau2 opened this issue Aug 22, 2016 · 2 comments
Assignees
@JustinBeckwith
Labels
type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.

Comments

@wilau2
Copy link

wilau2 commented Aug 22, 2016

I am using 12.3.0
directory.orgunits.get params.orgUnitPath does not seems to be encoded.
i tried to pass existing orgUnitPath : /Tennis Québec, this was the error :
Error: Org unit not found
If we have to encode param before calling api and that it is the expected behavior can we update documentation ?

Thanks!
@lukesneeringer
Copy link

@wilau2,
You can encode your URIs with encodeUriComponent. I agree that the docs should call this out (or it should just be done).

@lukesneeringer lukesneeringer added Priority: P2+ type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. labels Apr 12, 2017
@JustinBeckwith
Copy link
Contributor

Greetings! We started to encodeURIComponent for path parameters over in #1087. This should just work now. Let me know if you run into any issues!

@JustinBeckwith JustinBeckwith self-assigned this Feb 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JustinBeckwith JustinBeckwith

Labels
type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@lukesneeringer @JustinBeckwith @wilau2 and others