-
Notifications
You must be signed in to change notification settings - Fork 129
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
Base url note #135
Base url note #135
Conversation
@stewemetal All is okay now? :) |
@stewemetal ? :) Maybe @lejard-h ? :) |
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 good !
I will merge in master after this one to update the doc
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.
LGTM!
faq.md
Outdated
@@ -61,6 +62,20 @@ Request compressRequest(Request request) { | |||
Future<Response> postRequest(@Body() Map<String, String> data); | |||
``` | |||
|
|||
## Runtime baseUrl change | |||
|
|||
You may need to change the base URL of your network calls during runtime, for example, if you have to use different servers or routes dynamically in your app in case of a "regular" or a "paid" user. You can store the current server base url in your SharedPreferences (encrypt/decrypt it if needed) and use in interceptor like that: |
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.
Just a little adjustment:
"use in interceptor like that:" -> "use it in an interceptor like this:"
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.
Thanks, added your note! And squashed commits.
Runtime base url note update Update faq.md
No description provided.