require cycle

[suther]

This fixed the warning for "require cycle" on newer React-Versions

[msqar]

when is this going to be merged?? :S i hate those darn warnings.

[ThaJay]

why was that import even there if it is not used anyway?

[msqar]

does anyone have joltup's contact?? so we can ask him to merge this PR?

[el-lsan]

@Traviskn, @joltup Could you please consider this PR. 🙌

[alpha0010]

Looks like you may have trouble with these:

rn-fetch-blob/polyfill/XMLHttpRequest.js

Line 169 in 6c275ad

body = RNFetchBlob.wrap(blob.getRNFetchBlobRef())

rn-fetch-blob/polyfill/XMLHttpRequest.js

Lines 189 to 196 in 6c275ad

	this._task = RNFetchBlob
	.config({
	auto: true,
	timeout : this._timeout,
	increment : this._increment,
	binaryContentTypes : XMLHttpRequest.binaryContentTypes
	})
	.fetch(_method, _url, _headers, body)

[ThaJay]

In rn 59 it should be fine so we'd have to lock the next version tot that and up.
#183 (comment)

It could also be solved without rn 59 but is it worth the effort? In that case this pr would not suffice.

[mayconmesquita]

any updates here?

[Traviskn]

The RNFetchBlob import is used multiple times in this file, one call to RNFetchBlob.wrap and a second call to RNFetchBlob.config - This PR would break this file

[marcorm]

any updates?

[Traviskn]

The RNFetchBlob import is used multiple times in this file, one call to RNFetchBlob.wrap and a second call to RNFetchBlob.config - This PR would break this file