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: move other devices from 6865 to modernExtend.quirkPendingRequestTimeout #6874

Merged
merged 3 commits into from
Jan 11, 2024

Conversation

sjorge
Copy link
Sponsor Contributor

@sjorge sjorge commented Jan 10, 2024

Didn't need to add anything to the type, as it still takes number :) read over that yesterday.

@sjorge sjorge force-pushed the followup_6865 branch 2 times, most recently from 7e39e9f to 7df078c Compare January 10, 2024 08:30
@sjorge
Copy link
Sponsor Contributor Author

sjorge commented Jan 10, 2024

Wow I'm really not awake, as you change it in a later commit, and simply accepting both is gonna be a pain.

src/lib/modernExtend.ts Outdated Show resolved Hide resolved
@Koenkk Koenkk merged commit d284368 into Koenkk:master Jan 11, 2024
2 checks passed
@Koenkk
Copy link
Owner

Koenkk commented Jan 11, 2024

Looks good, thanks!

@sjorge sjorge deleted the followup_6865 branch January 11, 2024 18:45
@sjorge
Copy link
Sponsor Contributor Author

sjorge commented Jan 11, 2024

There is still some follow up work for zh I discovered #6865 (comment) @slugzero is gonna look at it. I might have time later next week too if they don't.

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.

None yet

2 participants