Skip to content

Comments

update server communication protocol to poll firebase for job status#30

Merged
ascibisz merged 1 commit intomainfrom
feature/use-updated-server-protocol
Jun 27, 2025
Merged

update server communication protocol to poll firebase for job status#30
ascibisz merged 1 commit intomainfrom
feature/use-updated-server-protocol

Conversation

@ascibisz
Copy link
Contributor

@ascibisz ascibisz commented Jun 24, 2025

Problem

We changed some things on the server side, so here are the corresponding changes on the client side!
Link to Server PR

Link to story or ticket

Solution

  • The Server PR talks in detail about the communication protocol changes, but here's the quick summary. Currently, the client sends a single request to the server to run a packing, and does not get a response until that packing is either done or failed. With this PR, the client sends a request to start a packing, and the server responds immediately with a job ID. Then, the client will continuously check firebase for job updates for that job ID until the job eventually succeeds or fails.
  • I also removed the "Run in Batch" button and the related batch stuff. We aren't going to be using batch going forward for this version of the client, so I think it's time to get rid of it.

@github-actions
Copy link

github-actions bot commented Jun 24, 2025

PR Preview Action v1.6.1
Preview removed because the pull request was closed.
2025-06-27 17:01 UTC

@ascibisz ascibisz merged commit 424ea3e into main Jun 27, 2025
1 check passed
@ascibisz ascibisz deleted the feature/use-updated-server-protocol branch June 27, 2025 17:01
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.

2 participants