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

Async port - draft #22

Closed
wants to merge 1 commit into from
Closed

Async port - draft #22

wants to merge 1 commit into from

Conversation

davidgraeff
Copy link
Owner

Missing:

  • Examples need to be ported
  • Blocking methods for the session type are missing
  • Formatting

@mardab
Copy link

mardab commented Apr 15, 2021

What about adding wasm support as well? I tried building with this branch, and the only problem seems to be mio, which does not support wasm yet.

@davidgraeff
Copy link
Owner Author

Wasm is not on my personal target list, but as soon as all dependencies (tokio, reqwest, oauth) support wasm this crate will do so as well :)

@mardab
Copy link

mardab commented Apr 20, 2021

Thanks, that would be nice, manually working with firestore/firebase is a lot of trouble.

@ncharchut
Copy link

Hi, is there any update for this particular PR? Would love to have async/await version of this crate, rather than blocking

@davidgraeff
Copy link
Owner Author

It's trickier than you think to provide an async and sync API without too much code duplication and without the async version to allocate. I'm haven't figured out all pieces yet.

@davidgraeff
Copy link
Owner Author

Async support has been merged with PR #34

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.

3 participants