-
Notifications
You must be signed in to change notification settings - Fork 479
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
feat: @farcaster/js should run in browsers #573
Comments
My two cents:
|
Supporting web browsers isn't an immediate problem, but in the medium term it lowers the barrier to entry for community members looking to make something using farcaster, which is good. I imagine some devs will want very basic access and choose to use the hub RPC for reading/writing and others will use more feature-rich APIs or build their own backends for indexing data. There's certainly an opportunity for community members to make their own indexing services and APIs on top of hubs for many use cases, similar to Alchemy's various APIs for ethereum. We should support both grpc and grpc-web simultaneously using a proxy (I made it work with Envoy, as you mentioned, though there may be other options too). So hub-to-hub communication will continue to use gRPC directly. |
we should see whether farcaster/js works by default in serverless platforms like vercel. if it does, we can punt this to a later milestone |
High-level goal:Create a new package called Work to be done:
|
It appears this is solved, yet the readme states that it is not I've supplied a PR to remove that! Let me know if this looks right 🙏 #1865 |
## Motivation Browser support was added via #573 but this documentation was not updated to reflex those changes ## Change Summary See above ^ ## Merge Checklist _Choose all relevant options below by adding an `x` now or at any time before submitting for review_ - [x] PR title adheres to the [conventional commits](https://www.conventionalcommits.org/en/v1.0.0/) standard - [ ] PR has a [changeset](https://github.com/farcasterxyz/hub-monorepo/blob/main/CONTRIBUTING.md#35-adding-changesets) - [ ] PR has been tagged with a change label(s) (i.e. documentation, feature, bugfix, or chore) - [ ] PR includes [documentation](https://github.com/farcasterxyz/hub-monorepo/blob/main/CONTRIBUTING.md#32-writing-docs) if necessary. - [x] All [commits have been signed](https://github.com/farcasterxyz/hub-monorepo/blob/main/CONTRIBUTING.md#22-signing-commits) <!-- start pr-codex --> --- ## PR-Codex overview This PR focuses on updating the README.md file in the hub-nodejs package. It includes changes related to error handling, required packages, and support for both Nodejs and browser environments. ### Detailed summary - Updated support information for Nodejs and browser environments. - Improved error handling with a monadic pattern. - Clarified the required packages for creating new messages. - Encoded fixed length data in byte formats for consistency. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex -->
What is the feature you would like to implement?
Get @farcaster/js compiling in browsers
Why is this feature important?
Single-page and stand alone client applications will be able to interface directly with Hubs without a server.
Will the protocol spec need to be updated??
No
Additional context
The text was updated successfully, but these errors were encountered: