Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Update poa module schema validation and endpoint #8967

Merged
merged 9 commits into from
Sep 18, 2023

Conversation

mitsuaki-u
Copy link
Contributor

@mitsuaki-u mitsuaki-u commented Sep 13, 2023

What was the problem?

This PR resolves #8944 and #8945, #8962 and #8965

How was it solved?

  • Updated endpoint
  • Schema validation for update authority command is done in PR
  • Use constants

How was it tested?

Check tests

@mitsuaki-u mitsuaki-u self-assigned this Sep 13, 2023
@codecov
Copy link

codecov bot commented Sep 13, 2023

Codecov Report

Merging #8967 (100f6b1) into release/6.1.0 (c659e39) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@              Coverage Diff               @@
##           release/6.1.0    #8967   +/-   ##
==============================================
  Coverage          84.35%   84.36%           
==============================================
  Files                650      650           
  Lines              24029    24022    -7     
  Branches            3491     3490    -1     
==============================================
- Hits               20269    20265    -4     
+ Misses              3760     3757    -3     
Files Changed Coverage
...k/src/modules/poa/commands/update_generator_key.ts ø
framework/src/modules/poa/stores/snapshot.ts ø
framework/src/modules/poa/utils.ts ø
...ork/src/modules/poa/commands/register_authority.ts 100.00%
framework/src/modules/poa/constants.ts 100.00%
framework/src/modules/poa/endpoint.ts 100.00%
framework/src/modules/poa/module.ts 100.00%

@mitsuaki-u mitsuaki-u changed the title Update poa endpoint getValidators to use schema validation Update poa module schema validation and endpoint Sep 14, 2023
Copy link
Contributor

@ishantiw ishantiw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, one comment that was missed I think

@ishantiw ishantiw merged commit 3a95f36 into release/6.1.0 Sep 18, 2023
11 checks passed
@ishantiw ishantiw deleted the 8944-poa-endpoint branch September 18, 2023 10:57
ishantiw added a commit that referenced this pull request Sep 20, 2023
* Code improvements and remove shuffleValidatorList from PoA

* Revert changes that has already been resolved in #8967 at update_generator_key

---------

Co-authored-by: !shan <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants