-
Notifications
You must be signed in to change notification settings - Fork 2k
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
changeOrigin not in documentation #851
Comments
I also just wasted several hours on this. |
I will definitely take a pull-request for it :) |
Sure. Who will put that extra line in the docs? Last one in line? I will do it when I have the time ;)
|
I'd do it but I have no idea where it should go. Any suggestions? The readme is kind of all over the place on doesn't really document anything. It just gives examples. Should it be its own example? |
Just mention it in the list of options with a short explanation. Thats enough
|
👍 also lost quite a bit of time until i found this in the docs. i'm not sure why it's not true by default! |
+1, changeOrigin should be defaultly set to true |
This cost me many hours to figure out....I might be stoopid, but more ppl like me could lost valuable time over this.
Why isn't this option enabled by default? It would serve 99% of the use cases out there.
The text was updated successfully, but these errors were encountered: