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

tx manager: staking on-chain interaction #34

Merged
merged 4 commits into from
Jan 8, 2024
Merged

tx manager: staking on-chain interaction #34

merged 4 commits into from
Jan 8, 2024

Conversation

hopeyen
Copy link
Collaborator

@hopeyen hopeyen commented Jan 5, 2024

Description

The most straight-forward way to make service available to protocol v1 involves deployments and allocations to an IPFS manifest that indicates their service. The possible units of deployments can be

  1. a file containing the manifest of a set of files; allocation must be renewed when the set of files changes or when allocation lifetime expires
  2. a file containing the server url; allocation must be renewed when url changes or when allocation lifetime expires

Regardless of the unit of deployment, they can be seen as an IPFS file. This PR consists of taking the IPFS file hash and

  1. generate unique allocation signer and allocation id
  2. allocation id proof for indexer ownership
  3. ipfs hash converted to bytes32 representation
  4. null array for metadata
  5. populate allocate transaction, estimate gas, and send transaction

NOTE for UI:

  1. require manual current epoch number
  2. wallet subcommand takes allocate action

Issue link (if applicable)

#4

Checklist

  • Are tests up-to-date with the new changes?
  • Are docs up-to-date with the new changes? (Open PR on docs repo if necessary)

@hopeyen hopeyen added size:large Large p1 High priority type:feature New or enhanced functionality labels Jan 5, 2024
@hopeyen hopeyen added this to the MVP milestone Jan 5, 2024
@hopeyen hopeyen self-assigned this Jan 5, 2024
Copy link

github-actions bot commented Jan 5, 2024

Pull Request Test Coverage Report for Build 7451640058

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+1.8%) to 49.18%

Totals Coverage Status
Change from base Build 7399839046: 1.8%
Covered Lines: 1290
Relevant Lines: 2623

💛 - Coveralls

@hopeyen hopeyen merged commit 47ccffa into dev Jan 8, 2024
5 checks passed
@hopeyen hopeyen deleted the hope/on-chain branch January 8, 2024 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p1 High priority size:large Large type:feature New or enhanced functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant