Skip to content
This repository has been archived by the owner. It is now read-only.

Change uses of XMLHttpRequest to fetch #2261

Closed
ianb opened this issue Mar 2, 2017 · 6 comments
Closed

Change uses of XMLHttpRequest to fetch #2261

ianb opened this issue Mar 2, 2017 · 6 comments

Comments

@ianb
Copy link
Contributor

@ianb ianb commented Mar 2, 2017

The fetch API is the more modern alternative to using XMLHttpRequest. We still have some uses of XMLHttpRequest in addon/webextension/ and these could be replaced. (We have some fetch examples too that you can use for reference.)

@pgadige
Copy link

@pgadige pgadige commented Mar 7, 2017

I'd like to work on this issue. Could you please assign it to me. Thank you.

@ianb
Copy link
Contributor Author

@ianb ianb commented Mar 7, 2017

@pgadige : I can't seem to assign it to you, but I'll assign it to myself so someone else doesn't take it (but I'll leave it to you to make the patch!)

@ianb ianb self-assigned this Mar 7, 2017
@pgadige
Copy link

@pgadige pgadige commented Mar 7, 2017

@ianb thank you very much for holding it up. Could you please try to assign it to me again, now?

@ianb
Copy link
Contributor Author

@ianb ianb commented Mar 22, 2017

Seems like there isn't activity on this, resetting assignment

@ianb ianb removed their assignment Mar 22, 2017
@ianb ianb self-assigned this Apr 10, 2017
@ianb
Copy link
Contributor Author

@ianb ianb commented Apr 10, 2017

I started this in the branch xmlhttprequest-to-fetch, but after doing this I'm feeling uncomfortable with the amount of logic being moved around at this late date. I don't think this work should be done until after our first release.

@ianb
Copy link
Contributor Author

@ianb ianb commented Feb 7, 2019

This issue is being closed (without being fixed) because we are discontinuing the Screenshots server.

@ianb ianb closed this Feb 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants