-
Notifications
You must be signed in to change notification settings - Fork 13
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
Use Promise type from web-promise library #24
Comments
Not sure there's any downside of that, the dependencies of (sorry I don't seem to have been watching the issues here) |
Would you like a PR for this @nwolverson ? |
Sorry I am away just now but yes I would take a pr for this yes
…On Sun, 22 May 2022, 16:55 James Brock, ***@***.***> wrote:
Would you like a PR for this @nwolverson <https://github.com/nwolverson> ?
—
Reply to this email directly, view it on GitHub
<#24 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAVEPSY2SQFTTEIZM35UJGTVLJKG7ANCNFSM5B7CYN2Q>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Shouldn't the |
Resolved by #30 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Do you have any interest in reusing the
Promise
type as exposed in theweb-promise
library? See:https://github.com/purescript-web/purescript-web-promise
A bit of work has gone in to the
Promise
representation in that library, and it may be useful to reuse the type here. I'd like to useaff-promise
in projects that are already using the type from that library, but unless I'm mistaken that's not currently possible.The text was updated successfully, but these errors were encountered: