Skip to content

Added generic AxiosAbortSignal TS interface to avoid importing AbortController polyfill#4229

Merged
jasonsaayman merged 14 commits intoaxios:masterfrom
DigitalBrainJS:feat/abort-controller-types
May 4, 2022
Merged

Added generic AxiosAbortSignal TS interface to avoid importing AbortController polyfill#4229
jasonsaayman merged 14 commits intoaxios:masterfrom
DigitalBrainJS:feat/abort-controller-types

Conversation

@DigitalBrainJS
Copy link
Copy Markdown
Collaborator

@DigitalBrainJS DigitalBrainJS commented Oct 29, 2021

Fixes #4124 by adding a generic AbortSignal interface.

@armandodlvr armandodlvr mentioned this pull request Nov 11, 2021
Closed
@DigitalBrainJS DigitalBrainJS mentioned this pull request Nov 30, 2021
Closed
@DigitalBrainJS
Copy link
Copy Markdown
Collaborator Author

DigitalBrainJS commented Nov 30, 2021

@jasonsaayman can this be reviewed and merged to avoid subsequent issues?

@vishalailani
Copy link
Copy Markdown

vishalailani commented Dec 2, 2021

@DigitalBrainJS / @jasonsaayman, i am also facing the same issue. is it merged ? if not please merge this as all checks have already passed.

@j4ys0n
Copy link
Copy Markdown

j4ys0n commented Dec 2, 2021

+1 can this be merged please? my build is failing because of the issue this fixes.

thank you!

@juditvarro
Copy link
Copy Markdown

juditvarro commented Dec 6, 2021 •
edited
Loading

+1
Could you please give an estimation @DigitalBrainJS / @jasonsaayman when can we expect that this will be merged? Thank you!

@DigitalBrainJS
Copy link
Copy Markdown
Collaborator Author

DigitalBrainJS commented Dec 7, 2021

@juditvarro Unfortunately, I can't provide an estimation as I'm not a member of the Axios team. The project is maintained by @jasonsaayman and @chinesedfan, so everything is up to them.

@corrjo
Copy link
Copy Markdown

corrjo commented Dec 9, 2021

I'll be the next to nag here. @jasonsaayman @chinesedfan 🙏

@swain
Copy link
Copy Markdown

swain commented Dec 27, 2021

+1 @jasonsaayman @chinesedfan

@nyalsadiq
Copy link
Copy Markdown

nyalsadiq commented Jan 31, 2022

Is there anything we can do to help get this merged soon? @jasonsaayman @chinesedfan

@jasonsaayman jasonsaayman merged commit 384b7e6 into axios:master May 4, 2022
@winklertomas winklertomas mentioned this pull request May 12, 2022
Merged
5 tasks
@jan-molak jan-molak mentioned this pull request Jun 2, 2022
Closed
@redpeacock78 redpeacock78 mentioned this pull request Oct 5, 2022
Merged
@girishjaju18 girishjaju18 mentioned this pull request Nov 22, 2022
Closed
@urugator urugator mentioned this pull request Nov 22, 2022
Merged
@DigitalBrainJS DigitalBrainJS mentioned this pull request Jan 31, 2024
Merged
@ayoshebby ayoshebby mentioned this pull request Apr 18, 2024
Merged
@katnova katnova mentioned this pull request May 19, 2024
Open
@rc042 rc042 mentioned this pull request May 19, 2024
Open
@nejidevelops nejidevelops mentioned this pull request May 19, 2024
Open
@katnova katnova mentioned this pull request May 19, 2024
Open
@BuloZB BuloZB mentioned this pull request May 20, 2024
Open
@ariel-enso ariel-enso mentioned this pull request May 20, 2024
Open
@nejidevelops nejidevelops mentioned this pull request May 20, 2024
Open
@katnova katnova mentioned this pull request May 21, 2024
Open
@olokotoh olokotoh mentioned this pull request May 21, 2024
Open
@cs-sagarmalve cs-sagarmalve mentioned this pull request Jun 1, 2024
Closed
@smashyalts smashyalts mentioned this pull request Jun 2, 2024
Open
@Hanae1963 Hanae1963 mentioned this pull request Jun 8, 2024
Open
@aravindvnair99 aravindvnair99 mentioned this pull request Jun 10, 2024
Closed
@q1blue q1blue mentioned this pull request Jun 17, 2024
Open
@gardnern gardnern mentioned this pull request Jun 18, 2024
Open
@aravindvnair99 aravindvnair99 mentioned this pull request Jul 2, 2024
Closed
@q1blue q1blue mentioned this pull request Jul 3, 2024
Open
@Stanislav1975 Stanislav1975 mentioned this pull request Aug 9, 2024
Open
@dinnysingh dinnysingh mentioned this pull request Aug 10, 2024
Open
@natelandon natelandon mentioned this pull request Sep 15, 2024
Open
@hbenali hbenali mentioned this pull request Sep 21, 2024
Merged
@fiftin fiftin mentioned this pull request Sep 28, 2024
Closed
@Zionsammy Zionsammy mentioned this pull request Oct 27, 2024
Open
This was referenced Oct 27, 2025
Closed
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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.

Usage of AbortController ist breaking for setup without DOM Library

8 participants

@DigitalBrainJS @vishalailani @j4ys0n @juditvarro @corrjo @swain @nyalsadiq @jasonsaayman