Skip to content

add fetch api #118

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

Closed
wants to merge 1 commit into from
Closed

add fetch api #118

wants to merge 1 commit into from

Conversation

plantain-00
Copy link
Contributor

@mhegazy
Copy link
Contributor

mhegazy commented Jun 10, 2016

👍

@mhegazy
Copy link
Contributor

mhegazy commented Jun 10, 2016

@zhengbli would this be autogenerated if we took a drop of the Edge definition files today?

@zhengbli
Copy link
Contributor

zhengbli commented Jun 15, 2016

@mhegazy Yes. The latest definite file in the redstone build has a GlobalFetch interface, and window implements it.

ok: boolean;
statusText: string;
headers: Headers;
clone(): Response;
Copy link
Contributor

@zhengbli zhengbli Jun 15, 2016

Choose a reason for hiding this comment

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

Request should also have a clone method that returns Request:
https://developer.mozilla.org/en-US/docs/Web/API/Request/clone

@mhegazy
Copy link
Contributor

mhegazy commented Jun 15, 2016

then we should just take a new drop, instead of hand authoring these definitions..

@zhengbli
Copy link
Contributor

As mentioned above, the new spec files that we haven't used yet contains the definition of fetch APIs, so it might be more consistent if we wait to take the drop. Although we should speed up the process to something like a monthly update. I'll talk to the Edge team to see if there are anything blocking that from happening.

Closing this issue for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants