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

add very basic web ui #25

Merged
merged 3 commits into from
Mar 12, 2024
Merged

add very basic web ui #25

merged 3 commits into from
Mar 12, 2024

Conversation

pavadeli
Copy link
Member

No description provided.

fix: update URL encoding for collection and document names to prevent issues with special characters
feat: add support for 'emulator-grpc' package and update 'emulator-ui' package version
refactor: Refactor code to improve modularity and readability

- Moved FirestoreEmulator implementation to a separate module for better organization.
Adds a new method `listen` to the `FirestoreProject` struct in order to listen for changes in Firestore documents. 

The `listen` method takes a stream of `ListenRequest` items as input and returns a stream of `ListenResponse` items. This method starts a listener for the given project and request stream, processing events received from the database in real-time.

This change introduces a new `Listener` struct to handle the listening functionality, managing the project, events, and target subscriptions. The `go` method of the `Listener` struct processes incoming requests and database events, handling errors and buffer overflows gracefully.

The listener is spawned as a separate task in the Tokio runtime to run concurrently with the application's main logic.
@pavadeli pavadeli enabled auto-merge March 12, 2024 16:33
@pavadeli pavadeli merged commit c3fc4de into master Mar 12, 2024
4 checks passed
@pavadeli pavadeli deleted the add-very-basic-web-ui branch March 12, 2024 16: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.

1 participant