Skip to content

feat(rs-dpp): introduce StateTransitionFactory#810

Merged
jawid-h merged 44 commits intov0.24-devfrom
feat/st-factory
Mar 15, 2023
Merged

feat(rs-dpp): introduce StateTransitionFactory#810
jawid-h merged 44 commits intov0.24-devfrom
feat/st-factory

Conversation

@jawid-h
Copy link
Copy Markdown
Contributor

@jawid-h jawid-h commented Feb 27, 2023

Issue being fixed or feature implemented

Introducing StateTransitionFactory for both rs-dpp and wasm-dpp

What was done?

  • added StateTransitionFactoy structure in rs-dpp
  • added StateTransitionBasicValidator structure to join different state transition basic validators into one
  • added StateTransitionFactory structure and bindings to wasm-dpp

How Has This Been Tested?

  • unit tests

Breaking Changes

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have made corresponding changes to the documentation

For repository code-owners and collaborators only

  • I have assigned this pull request to a milestone

@jawid-h jawid-h changed the title Feat/st factory feat(rs-dpp): introduce StateTransitionFactory Mar 9, 2023
@jawid-h jawid-h marked this pull request as ready for review March 9, 2023 09:58
@jawid-h jawid-h requested review from antouhou and shumkov as code owners March 9, 2023 09:58
Comment thread packages/rs-dpp/src/state_transition/state_transition_factory.rs
Comment thread packages/wasm-dpp/src/state_transition/errors/invalid_state_transition_error.rs Outdated
Comment thread packages/wasm-dpp/test/unit/stateTransition/StateTransitionFactory.spec.js Outdated
@jawid-h jawid-h requested a review from antouhou March 14, 2023 12:48
Copy link
Copy Markdown
Contributor

@antouhou antouhou left a comment

Choose a reason for hiding this comment

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

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants