Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

feat(fetch): patch the fetch API #119

Merged
merged 1 commit into from
May 20, 2015
Merged

feat(fetch): patch the fetch API #119

merged 1 commit into from
May 20, 2015

Conversation

vicb
Copy link
Contributor

@vicb vicb commented May 20, 2015

fixes #108

@vicb vicb mentioned this pull request May 20, 2015
Closed
@vicb
Copy link
Contributor Author

vicb commented May 20, 2015

@IgorMinar this seems to be working...
I haven't patched more APIs because:

  • APIs returning Promise are most of the time experimental - we might want to wait for more browsers to support fetch to test it (currently only Chrome supports it as of the SL versions),
  • I prefer that we agree on a first PR before jumping into this.

Note: #118 has an updated script to fetch more .idl and that will help

@IgorMinar
Copy link
Contributor

looks good! thanks

@vicb vicb merged commit 4d3d524 into angular:master May 20, 2015
@vicb vicb deleted the 0520-promiseFn branch May 20, 2015 21:02
@jeffbcross jeffbcross mentioned this pull request May 21, 2015
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

fetch api is not being intercepted by zones
2 participants
@vicb @IgorMinar