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

feat: show pending requests #149

Merged
merged 1 commit into from
Jun 2, 2022
Merged

Conversation

sdg9
Copy link
Collaborator

@sdg9 sdg9 commented Jun 2, 2022

image
image

screenshots use mock request and response data with same GUID so URL differs (not realistic scenario) but concept is request will show, then when fulfilled with response it will update based on matching GUID.

Specifically when we receive a recording item we "add it", if GUID is new it's added. Then when response is received, since GUID already exists, it updates the record in the reducer.

If for some reason response comes through with no existing entry in reducer that is fine it's added, idea is if no GUID match we add, if match we update the record.

@nx-cloud
Copy link

nx-cloud bot commented Jun 2, 2022

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 9bc6df8. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch


✅ Successfully ran 2 targets

Sent with 💌 from NxCloud.

@sdg9 sdg9 force-pushed the sdg9/showRequestInFlight branch from e531a94 to 9bc6df8 Compare June 2, 2022 03:51
@sdg9 sdg9 merged commit d33f389 into caribou-crew:main Jun 2, 2022
@sdg9 sdg9 deleted the sdg9/showRequestInFlight branch June 2, 2022 13:10
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