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

[PVM] Add kyb verified state, allow kyb verified to propose add member #355

Merged
merged 1 commit into from
Jul 25, 2024

Conversation

evlekht
Copy link
Member

@evlekht evlekht commented Jul 23, 2024

Why this should be merged

This PR adds AddressStateKYBVerified, its bit 34.
AddressStateKYBVerified bit can be set by AddressStateRoleKYCAdmin.
This bit allows to create addMemberProposals.

How this works

AddressStateKYBVerified is added to BerlinPhase bits set, so its treated as valid bit starting from BerlinPhase.
PR adds corresponding test cases for AddressStateTx and for addMemberProposal.

How this was tested

By existing unit tests.

Additional references

Original PR based on cortina-19 dev
#340

@evlekht evlekht force-pushed the cortina-15-dev/kyb-addr-state branch 3 times, most recently from 098b9dc to ac55cf6 Compare July 24, 2024 10:44
@evlekht evlekht changed the base branch from evlekht/new-dev-resulting to cortina-15-dev/kopernikus-genesis July 24, 2024 11:07
havan
havan previously approved these changes Jul 24, 2024
Copy link
Member

@havan havan left a comment

Choose a reason for hiding this comment

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

Looks consistent with the original PR.

@havan havan added the checked Temporary label for Cortina-15 revert label Jul 24, 2024
@evlekht evlekht force-pushed the cortina-15-dev/kopernikus-genesis branch 2 times, most recently from 0f7232a to 89b58eb Compare July 25, 2024 13:18
Base automatically changed from cortina-15-dev/kopernikus-genesis to dev July 25, 2024 13:21
@evlekht evlekht dismissed havan’s stale review July 25, 2024 13:21

The base branch was changed.

@evlekht evlekht force-pushed the cortina-15-dev/kyb-addr-state branch from ac55cf6 to c71e284 Compare July 25, 2024 13:23
@evlekht evlekht marked this pull request as ready for review July 25, 2024 13:23
@evlekht evlekht merged commit 2144505 into dev Jul 25, 2024
2 of 16 checks passed
@evlekht evlekht deleted the cortina-15-dev/kyb-addr-state branch July 25, 2024 13:23
evlekht added a commit that referenced this pull request Aug 2, 2024
#355)

(cherry picked from commit 2144505)
(cherry picked from commit 0e19378bccbcf43c4ffa124a992afa29d22ecd1a)
evlekht added a commit that referenced this pull request Aug 2, 2024
#355)

(cherry picked from commit 2144505)
(cherry picked from commit 0e19378bccbcf43c4ffa124a992afa29d22ecd1a)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
checked Temporary label for Cortina-15 revert
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants