Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

[v2] Disperser GetBlobStatus endpoint #889

Merged
merged 1 commit into from
Nov 15, 2024
Merged

[v2] Disperser GetBlobStatus endpoint #889

merged 1 commit into from
Nov 15, 2024

Conversation

ian-shim
Copy link
Contributor

Why are these changes needed?

Implementation of GetBlobStatus endpoint on v2 dispersal server

Checks

  • I've made sure the tests are passing. Note that there might be a few flaky tests, in that case, please comment that they are not relevant.
  • I've checked the new test coverage and the coverage percentage didn't drop.
  • Testing Strategy
    • Unit tests
    • Integration tests
    • This PR is not tested :(

@ian-shim ian-shim marked this pull request as ready for review November 13, 2024 07:10
mooselumph
mooselumph approved these changes Nov 15, 2024
View reviewed changes
disperser/apiserver/get_blob_status_v2.go Show resolved Hide resolved
disperser/apiserver/get_blob_status_v2.go Outdated Show resolved Hide resolved
@ian-shim ian-shim force-pushed the v2-blob-status-endpoint branch from e09336f to 7e3e517 Compare November 15, 2024 05:22
@ian-shim ian-shim merged commit d0dfd2c into Layr-Labs:master Nov 15, 2024
6 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@cody-littley cody-littley cody-littley approved these changes

@mooselumph mooselumph mooselumph approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ian-shim @mooselumph @cody-littley