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(ph4): agent and api #4323

Merged
merged 7 commits into from
Sep 25, 2023
Merged

feat(ph4): agent and api #4323

merged 7 commits into from
Sep 25, 2023

Conversation

nugaon
Copy link
Member

@nugaon nugaon commented Sep 18, 2023

Adding phase 4 related agent and API changes.

This codebase could produce valid phase4 claim arguments.

The code needs refactor after merging but for the sake of straightforward changes, just substantial minimal additions and modifications were made.

The PR is based on the work of #4059 and #3823 PRs

remaining tasks

Checklist

  • I have read the coding guide.
  • My change requires a documentation update, and I have done it.
  • I have added tests to cover my changes.
  • I have filled out the description and linked the related issues.

Description

Open API Spec Version Changes (if applicable)

Motivation and Context (Optional)

Related Issue (Optional)

Screenshots (if appropriate):

@nugaon nugaon marked this pull request as ready for review September 19, 2023 12:04
@nugaon nugaon requested a review from zelig September 19, 2023 12:04
Base automatically changed from feat/phase4-bmt to master September 21, 2023 13:02
pkg/bmt/bmt.go Outdated Show resolved Hide resolved
pkg/bmt/proof.go Outdated Show resolved Hide resolved
pkg/storer/sample.go Show resolved Hide resolved
pkg/swarm/hasher.go Outdated Show resolved Hide resolved
@nugaon nugaon changed the base branch from master to feat/phase4 September 25, 2023 10:22
@nugaon nugaon merged commit f89ad31 into feat/phase4 Sep 25, 2023
13 checks passed
@nugaon nugaon deleted the feat/ph4 branch September 25, 2023 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants