Skip to content
This repository has been archived by the owner on Jun 7, 2020. It is now read-only.

[FIX] 2.3.0 hotfix1 #1345

Merged
merged 3 commits into from
Jun 6, 2018
Merged

[FIX] 2.3.0 hotfix1 #1345

merged 3 commits into from
Jun 6, 2018

Conversation

luciofm
Copy link
Contributor

@luciofm luciofm commented Jun 6, 2018

Fixes for 2.3.0


import javax.inject.Qualifier

/**
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you can't now, we'll remove that later 👍

import java.util.concurrent.TimeUnit

/**
* An OkHttp interceptor which logs request and response information. Can be applied as an
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

documentation...
💯

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💯 copied from OkHttp logger... It is just a modified/kotlinfied version to remove the auth token from the logs

try {
customTabsIntent.launchUrl(context, url)
} catch (ex: Exception) {
Timber.d(ex, "Unable to launch URL")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@philipbrito philipbrito merged commit 1182d85 into master Jun 6, 2018
@philipbrito philipbrito deleted the release/2.3.0-hotfix1 branch June 6, 2018 17:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants