Skip to content
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

fix: patch the angular browser.js to allow {crypto: true, stream: true} #3136

Merged
merged 1 commit into from
Oct 16, 2019
Merged

fix: patch the angular browser.js to allow {crypto: true, stream: true} #3136

merged 1 commit into from
Oct 16, 2019

Conversation

joshstevens19
Copy link
Contributor

@joshstevens19 joshstevens19 commented Oct 16, 2019

fix 2 issues which are in the list for stabilize Web3 1.x - #3070

This fixes:

#2262

It also knocks out this issue - #2336 as its not relevant in 1.x.

without this on apps like angular web3 1.x wont work without you going into the node_modules of angulars and changing this file yourself.

@joshstevens19 joshstevens19 added the 1.x 1.0 related issues label Oct 16, 2019
@nivida nivida added the Bug Addressing a bug label Oct 16, 2019
@nivida nivida merged commit ba56fc6 into web3:1.x Oct 16, 2019
@joshstevens19 joshstevens19 deleted the fix/angular/2262 branch October 16, 2019 15:10
@coveralls
Copy link

Coverage Status

Coverage remained the same at 84.829% when pulling 3561368 on joshstevens19:fix/angular/2262 into 8ac5c0e on ethereum:1.x.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.x 1.0 related issues Bug Addressing a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants