Skip to content

Conversation

@jordy25519
Copy link
Contributor

@jordy25519 jordy25519 commented Oct 12, 2020

This prevents storage changes from persisting if something fails during intermediate operations (new feature in 2.0)

@jordy25519 jordy25519 assigned aliXsed and unassigned aliXsed Oct 12, 2020
@jordy25519 jordy25519 requested a review from aliXsed October 12, 2020 23:05
Copy link

@aliXsed aliXsed left a comment

Choose a reason for hiding this comment

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

LGTM!

@jordy25519 jordy25519 merged commit 42453c7 into update/2.0.0 Oct 12, 2020
@jordy25519 jordy25519 deleted the fix/execute-trade branch October 12, 2020 23:46
aliXsed pushed a commit that referenced this pull request Oct 22, 2020
* update/2.0.0:
  Slashed CENNZ to treasury/reporter (#299)
  Update plug commit to get correct WeightInfo for prmls (#300)
  Bonding is enabled by default but can be blocked by root (#298)
  Chore/config runtime fee (#296)
  Config attestation in the runtime (#295)
  Add transaction payment rpc to cli (#294)
  Update/fix test build (#293)
  Update/2.0.0 staking update (#292)
  Make execute_trade atomic (#291)
  Update CENNZX (#287)
  Update/transaction payment (#290)
  Reinstate dev + nikau genesis builds (#289)
  Build Sylo modules and its tests and make the tests pass (#288)
  Fmt
  Undo WIP sylo changes
  Client compile and sync
  Cargo deps fixed
  Format
  CENNZX compiles
  [WIP] fixing cennzx

# Conflicts:
#	runtime/src/impls.rs
#	runtime/src/lib.rs
#	runtime/tests/staking_reward.rs
aliXsed pushed a commit that referenced this pull request Oct 22, 2020
* Update cennznet to be compatible with and use Plug 2.0

* Make execute_trade atomic (#291)

* Remove contract tests

* Bonding is enabled by default but can be blocked by root (#298)

* Slashed CENNZ to treasury/reporter (#299)

* Use negative imbalance with correct asset ID

* Use automated slash and reported slash in tests

* downgrade wabt version: paritytech/substrate#7042

* Ignore tests that are being worked on as part of the "configuring runtime v37" task

Co-authored-by: Jordan Beauchamp <[email protected]>
Co-authored-by: KarishmaBothara <[email protected]>
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.

4 participants