uber / rides-java-sdk

Uber Rides Java SDK (beta)
https://developer.uber.com/docs
MIT License
104 stars 57 forks source link

default to Environment.SANDBOX #2

Closed MarianoTucat closed 8 years ago

MarianoTucat commented 8 years ago

I think Environment.SANDBOX should be used by default, when no environment is set. I have just made a real request by mistake (my mistake) of omitting the environment. That would not have happened If it was set by default to SANDBOX.

noonhub commented 8 years ago

Thanks for the feedback. We are taking into consideration and will let you know what design we land on. Regardless of that outcome, I think that clearer and more prominent documentation around the exact behavior will help to avoid future confusion.

itstexter commented 8 years ago

Fixed with release of v0.2.0, went with explicit declaration of Environment