Skip to content
This repository was archived by the owner on Feb 22, 2018. It is now read-only.

fix(Http): Fix NoSuchMethodError in Http #1075

Merged
merged 1 commit into from
Jun 4, 2014

Conversation

mvuksano
Copy link
Contributor

Fix NoSuchMethodError in Http when cache set to true and update documentation about cache usage.

Closes #1066

@mvuksano mvuksano added cla: yes and removed cla: no labels May 28, 2014
* - cache: Boolean or [Cache]. If true, the default cache will be used.
* - cache: Boolean or [Cache]. If true or not specified at all, the default cache will be used.
* If false, no cache will be used. If object of type [Cache] is provided, that object will
* be used as cache.
* - timeout: deprecated
Copy link
Contributor

Choose a reason for hiding this comment

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

What about a bullet list here

  - if true/null, the [defaults.cache] will be used,
  - if false ...
  - if object of type [Cache] ...

[defaults.cache] better describes what is used (and it could actually not be a cache, ie null)

Copy link
Contributor

Choose a reason for hiding this comment

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

Otherwise LGTM

@jbdeboer
Copy link
Contributor

jbdeboer commented Jun 3, 2014

Looks good to merge

@mhevery mhevery merged commit 5b48332 into dart-archive:master Jun 4, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

Successfully merging this pull request may close these issues.

Http documentation doesn't match code functionality
4 participants