Skip to content

@transaction - Transaction management decorator#1067

Closed
KPrasch wants to merge 58 commits intonucypher:alhambra-verdefrom
KPrasch:del-mar
Closed

@transaction - Transaction management decorator#1067
KPrasch wants to merge 58 commits intonucypher:alhambra-verdefrom
KPrasch:del-mar

Conversation

@KPrasch
Copy link
Member

@KPrasch KPrasch commented Jun 11, 2019

No description provided.

KPrasch and others added 30 commits June 10, 2019 11:38
@KPrasch @cygnusv
Almost all of checksum_public_address -> checksum_address
4f64033
@KPrasch @cygnusv
Testing decentralized grant, Adds missing datetime to policy creation…
d7e2ddf 
… tests.
@KPrasch @cygnusv
Generic and specific character configuration testing
42b70db
@KPrasch @cygnusv
Remove expiration date from create policy entry points
5c09bc0
@KPrasch @cygnusv
Alice restores default policy values from configuration file
296af42
@KPrasch @cygnusv
Refining blockchian sync management for internal geth processes
1479c5a
@KPrasch @cygnusv
Restores skipped stake division test on MinerAgent
e6c1c37
@KPrasch @cygnusv
Alice view command is a little more eager.
0cece4d
@KPrasch @cygnusv
Creating a code path for... Bob. Aviods ethereum client logic during …
b495289 
…CLI init command.
@KPrasch @cygnusv
Use a common bootstrapping function for making cli characters
cecb7f1
@KPrasch @cygnusv
Gives PolicyAuthor an economics instance
ea4ff7b
@KPrasch @cygnusv
Passing decentralized grant CLI lifecycle test
bd8d2b1
@KPrasch @cygnusv
Thin out web3 client syncing, Use configuration classes for default c…
5d55ce0 
…ontroller ports
@KPrasch @cygnusv
Accept alice configuration policy defaults at init-time by CLI flags.
e2d449a
@KPrasch @cygnusv @derekpierre
Respond to RFCs in PR nucypher#984; Apply suggestions from code review
6129dcf 
Co-Authored-By: David Núñez <david@nucypher.com>
Co-Authored-By: Derek Pierre <derek.pierre@gmail.com>
@KPrasch @cygnusv
Cleanup policy and arrangement value code paths - Fixes Issue nucyphe…
7c8b986 
…r#1015 - BlockchainPolicy value must be greater than 0.
@KPrasch @cygnusv
Style and convention improvements found while debugging failed reward…
e80455d 
… collection test
@KPrasch @cygnusv
Fixes miscalculated first period reward rate (See nucypher#1063), Imp…
b72f681 
…roves depth of reward collection integration testing
@KPrasch @cygnusv
Sanity checks for testerchain
1983fa0
@KPrasch @cygnusv
Merge pull request nucypher#984 from KPrasch/casual-geth
f5d7434 
Casual Alice with Policy and Payment Defaults
@cygnusv
Added worker role, excluded minting and activity confirmation from lo…
d0ecf82 
…ck method
@cygnusv
Fix agents tests
71dd606
@cygnusv
Fixed small issues, added setWorker to UserEscrow
1b19ca3
@cygnusv
Added test for setting worker
2c5dc11
@cygnusv
Fixed agent tests
5dd3f22
@cygnusv
Moved minimum worker periods to constructor
8974acb
@cygnusv
Small changes in docs
4cfc5d9
@cygnusv
Added way to slash intermediary contracts
6c9bf22
@cygnusv
Tests for slashing worker
9a59391
@cygnusv
Increase gas limit again
c895348
szotov and others added 25 commits June 10, 2019 11:40
@cygnusv
miner->staker in tests and docs
7218564
@cygnusv
Apply suggestions from code review
cad9103
@KPrasch @cygnusv
MinerEscrow -> StakingEscrow, and MiningAdjudicator -> Adjudicator in…
d0ae0b0 
… agency, and in-test
@KPrasch @cygnusv
Goodbye, Three Agents. Hello, Agency.
446c37b
@KPrasch @cygnusv
Understand Staker and Staking to be separate ideas.
6fd2d99
@cygnusv
Leftovers: Miner --> Staker, MiningAdjudicator --> Adjudicator, etc.
697bed2
@cygnusv
Removing some re-emerging garbage, probably from a rebase
de1f01c
@cygnusv
Fix test for emission of unknown LL version warnings. Closes nucypher…
6e616e6 
…#1039
@cygnusv
Fix name of StakingEscrowAgent. It was StakingEscrow
e2a3ec9
@cygnusv
Missing renamings to "worker" and "staker" in agents.py
42df7cd
@cygnusv
Staker doesn't confirm activity anymore
0c4a029
@cygnusv
First stub of Worker class, extracted from Staker
eb25113
@cygnusv
Simplify StakingEscrow.confirmActivity(): only direct calls from Worker
83927ce
@cygnusv
Stakers will need their own test ETH accounts, different than Ursulas'
e506a7a
@cygnusv
Ursula.rest_information()[0] --> Ursula.rest_interface()
588562d
@cygnusv
Ursula character is not a Staker anymore, but a Worker
939111e
@cygnusv
Change "by" to "from" in getWorkerByStaker and getStakerByWorker
be66b05
@cygnusv
StakingEscrowAgent API for checking staker-worker relationship
4337c63
@cygnusv
Worker validation pursuant to consensus on nucypher#1003. Closes nucy…
27c9c80 
…pher#1033.
@cygnusv
New stakers fixture, consistent with blockchain_ursulas
abd2d50
@cygnusv
Introduce simple PRE lifecycle test with blockchain ursulas
13a731f
@cygnusv
Touchups here and there
40ba189
@cygnusv
Fix test_staking_escrow_agent tests
8a26dd5
@KPrasch
Introduces @transaction decorator for agent subclass methods
9fa0823
@KPrasch
Implement @transaction on agency methods
3bbd9a4
@cygnusv cygnusv force-pushed the alhambra-verde branch 2 times, most recently from 4eca13d to b1572a7 Compare June 11, 2019 08:05
@KPrasch
Copy link
Member Author

KPrasch commented Jun 11, 2019

Commits included in PR #1058

@KPrasch KPrasch closed this Jun 11, 2019
@KPrasch KPrasch mentioned this pull request Jun 18, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@KPrasch @cygnusv