Skip to content

[EPIC] Coordinator Alpha 13 Support#3226

Merged
derekpierre merged 88 commits intodevelopmentfrom
contracts-alpha-13
Sep 20, 2023
Merged

[EPIC] Coordinator Alpha 13 Support#3226
derekpierre merged 88 commits intodevelopmentfrom
contracts-alpha-13

Conversation

@derekpierre
Copy link
Copy Markdown
Member

@derekpierre derekpierre commented Sep 12, 2023

KPrasch and others added 30 commits September 7, 2023 18:27
This commit is aimed to update test to the following changes on
nucypher-contracts repo:
  - Update SimplePREApplication with TACoApplication
  - Replace ThresholdStakingForPREApplicationMock contract with
    ThresholdStakingForTACoApplicationMock
  - Use the new Coordinator contract
  - Add the deployment of StakeInfo contract

The use of these new contracts made necessary the following changes:
  - Add the deployment of a new Token for ritual fees: RitualToken. This
    contract has been added as local deployment.
  - Add the deployment of local contract ConditionNFT

Additionally, this commit updates the key character for arguments in
ape-config.yaml. From '::variable::' to <variable>, which results in a
more legible code.

Finally, this commit solves some linting and formatting issues.

Co-authored-by: LunarBytes <kieran@nucypher.com>
…public keys.

Move G2Point out of Ritual in CoordinatorAgent.
…bject.

MockCoordinatorAgent simply uses a class method to determine threshold (allows for both simple/precomputed) to still be tested in integration tests.
… tests accordingly; this way we don't care about fee values.
derekpierre and others added 7 commits September 14, 2023 21:46
…n only be stored in the json configuration file as dictionary keyed by the string version of chain id.

The chain id is only stored in the configuration file as a string even though it is used in python as an integer.
When the UrsulaConfiguration is serialized to json the integer (chain id) is automatically stringified to be valid json. Therefore, when deserializing the configuration i.e. reading it in, chain must be read in as a string first then converted to integer.
…o ensure proper changes are enacted.

Migration from v4->v5 missed deprecation of "db_filepath" configuration parameter.
Add tests for configuration migration.
@derekpierre derekpierre changed the title [WIP] [EPIC] Coordinator Alpha 13 Support [EPIC] Coordinator Alpha 13 Support Sep 20, 2023
@derekpierre derekpierre marked this pull request as ready for review September 20, 2023 17:05
@derekpierre derekpierre merged commit 7098933 into development Sep 20, 2023
@derekpierre derekpierre deleted the contracts-alpha-13 branch September 20, 2023 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Completed

Development

Successfully merging this pull request may close these issues.

3 participants