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.

Sign up for GitHub

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

Jump to bottom

Revert "MN-574-2" #78

Merged
merged 2 commits into from
Sep 7, 2020
Merged

Revert "MN-574-2" #78

merged 2 commits into from
Sep 7, 2020

Conversation

pavlushev
Copy link
Contributor

Reverts #74

@pavlushev
Revert "MN-574-2 (#74)"
6b2d892 
This reverts commit dbc9e6c.
@codecov-commenter
Copy link

codecov-commenter commented Sep 7, 2020 •
edited
Loading

Codecov Report

Merging #78 into release/1.11 will increase coverage by 0.29%.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff                @@
##           release/1.11      #78      +/-   ##
================================================
+ Coverage         13.95%   14.24%   +0.29%     
================================================
  Files                 8        7       -1     
  Lines              1197     1123      -74     
================================================
- Hits                167      160       -7     
+ Misses             1017      950      -67     
  Partials             13       13
Impacted Files Coverage Δ
...cation/builtin/contract/deposit/deposit.wrapper.go 0.00% <0.00%> (ø)
...ication/builtin/contract/deposit/linear_vesting.go
...et/application/builtin/contract/deposit/deposit.go 2.48% <0.00%> (+0.46%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update df73e5d...c0ddad2. Read the comment docs.

@pavlushev pavlushev merged commit 8e3fe17 into release/1.11 Sep 7, 2020
@pavlushev pavlushev deleted the revert-74-MN-574-2 branch September 7, 2020 15:30
pavlushev added a commit that referenced this pull request Oct 16, 2020
@pavlushev @uhzxl
@megge-dream @MikhailPopov @Hirama @pavel-moskovkin @sdmironov @Tsovak
NOISSUE: release/1.11-> release/1.12 (#87)
f3aa5ba 
* MN-574-2 (#74)

* MN-581: create method deposit.transferToDeposit

* MN-581: create method deposit.transferToDeposit

* MN-591: add `deposit.createFund` method (#39)

* PENV-346: add hash to full pulse

* MN-591: add `deposit.createFund` method

* MN-591: refactor method names and remove redundant constructor parameter

* MN-591: switch to master

* MN-591: cleanup after rebase

Co-authored-by: Maria Zubareva <maria.zubareva@insolar.io>

* MN-581: create method deposit.transferToDeposit

* resolve conflict

* resolve conflict

* MN-636: add method `account.transferToDeposit` (#42)

* MN-636: add method `account.transferToDeposit`

* MN-636: rename `TransferToDeposit` to `ReallocateToDeposit`

* MN-636: add balance checks to functest

* MN-584: add linear vesting type (#47)

* MN-583: add method `deposit.create` (#53)

* MN-651: split spec files (#46)

* MN-651: split spec files

* MN-651: return temporary admin methods in api-exported.yaml

* MN-651: reset api-exported.yaml to v1.1.1 (#48)

* MN-662: fix TestMigrationDaemonTransferDeposit

* MN-662: fix TransferDeposit tests

* MN-583: add method `deposit.create`

* MN-583: revert fund_test.go

* MN-583: fix typo in daemon_confirm.go name

* MN-583: comply note about error text checking

* MN-583: fix parameters order for compatibility with observer

* MN-583: fix note about error type assertion

Co-authored-by: Mikhail Popov <mikhail.popov@insolar.io>

* MN-583: add double deposit creation check (#57)

* NOISSUE: clarify deposit.Confirm (#58)

* NOISSUE: clarify deposit.Confirm

* NOISSUE: remove unnecessary confusing test

* NOISSUE: fix note about toAmounts method

* MN-588: update migration procedure (#65)

* MN-588: update migration procedure

* MN-588: skip TestDepositCreate

* Update application/functest/test_utils.go

Co-authored-by: Zubareva Maria <maria.zubareva@insolar.io>

Co-authored-by: Zubareva Maria <maria.zubareva@insolar.io>

* MN-652: calculate available amount for additional deposit (#70)

* MN-589: utility app

* MN-751: fill linear vesting params from default vesting params config (#73)

Co-authored-by: Andrey Pavlushev <andrey.pavlushev@insolar.io>
Co-authored-by: Maria Zubareva <maria.zubareva@insolar.io>
Co-authored-by: Andrey Pavlushev <pavlushev@users.noreply.github.com>
Co-authored-by: Mikhail Popov <mikhail.popov@insolar.io>
Co-authored-by: Andrey Zolin <crrrr@mail.ru>

* MN-740: add insolar v1.8.0 with an exporter API rate limiter

* MN-761: use TransferToDeposit request reference as txID (#76)

* MN-740: add fixed insolar v1.8.1

* Revert "MN-761: use TransferToDeposit request reference as txID (#76)" (#77)

This reverts commit 90a2e08.

* Revert "MN-574-2 (#74)" (#78)

This reverts commit dbc9e6c.

* MN-771: add the insolar v1.8.2 with an optional rate-limiting

* OPS-354 run e2e tests on pull requests (#82)

* OPS-354 e2e

* OPS-354 move overlay to gotops

* OPS-354 debug

* OPS-354: add private registry secret to pull go-autotest image

* OPS-354: fix private registry secret

* OPS-354 download go-test-parser.zip

* OPS-354 fix execute binary

* OPS-354 fix execute binary

* OPS-354 tmp

* OPS-354 fix execute binary

* OPS-354 tmp

* OPS-354 Download go-test-parser and execute

* OPS-354 provide BUILD_VERSION to fix the autotest

* OPS-354 fix quote

* empty commit

* OPS-354 proper path to autotests

* OPS-354 update gitops branch to REL-57

* OPS-354 rename package to 'autotests-pod'

* OPS-354 update go-autotests branch to master

* OPS-354 go-test-parser v1.0.4 check

* OPS-354 go-test-parser v1.0.4 check

* OPS-354 revert step condition

Co-authored-by: Sergei Mironov <sergei.mironov@insolar.io>

Co-authored-by: Alexey Zhunin <zhunin.a.u@gmail.com>
Co-authored-by: Maria Zubareva <maria.zubareva@insolar.io>
Co-authored-by: Mikhail Popov <mikhail.popov@insolar.io>
Co-authored-by: Andrey Zolin <crrrr@mail.ru>
Co-authored-by: Mikhail Popov <53470434+MikhailPopov@users.noreply.github.com>
Co-authored-by: Pavel Moskovkin <34027831+pavel-moskovkin@users.noreply.github.com>
Co-authored-by: Sergei Mironov <sergei.mironov@insolar.io>
Co-authored-by: Tsovak Sahakyan <tsovak@bk.ru>
pavlushev added a commit that referenced this pull request Oct 16, 2020
@MikhailPopov @uhzxl
@pavlushev @megge-dream @Hirama @pavel-moskovkin @sdmironov
PENV-680: remove vendor, update go (#83)
580deef 
* MN-574-2 (#74)

* MN-581: create method deposit.transferToDeposit

* MN-581: create method deposit.transferToDeposit

* MN-591: add `deposit.createFund` method (#39)

* PENV-346: add hash to full pulse

* MN-591: add `deposit.createFund` method

* MN-591: refactor method names and remove redundant constructor parameter

* MN-591: switch to master

* MN-591: cleanup after rebase

Co-authored-by: Maria Zubareva <maria.zubareva@insolar.io>

* MN-581: create method deposit.transferToDeposit

* resolve conflict

* resolve conflict

* MN-636: add method `account.transferToDeposit` (#42)

* MN-636: add method `account.transferToDeposit`

* MN-636: rename `TransferToDeposit` to `ReallocateToDeposit`

* MN-636: add balance checks to functest

* MN-584: add linear vesting type (#47)

* MN-583: add method `deposit.create` (#53)

* MN-651: split spec files (#46)

* MN-651: split spec files

* MN-651: return temporary admin methods in api-exported.yaml

* MN-651: reset api-exported.yaml to v1.1.1 (#48)

* MN-662: fix TestMigrationDaemonTransferDeposit

* MN-662: fix TransferDeposit tests

* MN-583: add method `deposit.create`

* MN-583: revert fund_test.go

* MN-583: fix typo in daemon_confirm.go name

* MN-583: comply note about error text checking

* MN-583: fix parameters order for compatibility with observer

* MN-583: fix note about error type assertion

Co-authored-by: Mikhail Popov <mikhail.popov@insolar.io>

* MN-583: add double deposit creation check (#57)

* NOISSUE: clarify deposit.Confirm (#58)

* NOISSUE: clarify deposit.Confirm

* NOISSUE: remove unnecessary confusing test

* NOISSUE: fix note about toAmounts method

* MN-588: update migration procedure (#65)

* MN-588: update migration procedure

* MN-588: skip TestDepositCreate

* Update application/functest/test_utils.go

Co-authored-by: Zubareva Maria <maria.zubareva@insolar.io>

Co-authored-by: Zubareva Maria <maria.zubareva@insolar.io>

* MN-652: calculate available amount for additional deposit (#70)

* MN-589: utility app

* MN-751: fill linear vesting params from default vesting params config (#73)

Co-authored-by: Andrey Pavlushev <andrey.pavlushev@insolar.io>
Co-authored-by: Maria Zubareva <maria.zubareva@insolar.io>
Co-authored-by: Andrey Pavlushev <pavlushev@users.noreply.github.com>
Co-authored-by: Mikhail Popov <mikhail.popov@insolar.io>
Co-authored-by: Andrey Zolin <crrrr@mail.ru>

* MN-740: add insolar v1.8.0 with an exporter API rate limiter

* MN-761: use TransferToDeposit request reference as txID (#76)

* MN-740: add fixed insolar v1.8.1

* Revert "MN-761: use TransferToDeposit request reference as txID (#76)" (#77)

This reverts commit 90a2e08.

* Revert "MN-574-2 (#74)" (#78)

This reverts commit dbc9e6c.

* MN-771: add the insolar v1.8.2 with an optional rate-limiting

* OPS-354 run e2e tests on pull requests (#82)

* OPS-354 e2e

* OPS-354 move overlay to gotops

* OPS-354 debug

* OPS-354: add private registry secret to pull go-autotest image

* OPS-354: fix private registry secret

* OPS-354 download go-test-parser.zip

* OPS-354 fix execute binary

* OPS-354 fix execute binary

* OPS-354 tmp

* OPS-354 fix execute binary

* OPS-354 tmp

* OPS-354 Download go-test-parser and execute

* OPS-354 provide BUILD_VERSION to fix the autotest

* OPS-354 fix quote

* empty commit

* OPS-354 proper path to autotests

* OPS-354 update gitops branch to REL-57

* OPS-354 rename package to 'autotests-pod'

* OPS-354 update go-autotests branch to master

* OPS-354 go-test-parser v1.0.4 check

* OPS-354 go-test-parser v1.0.4 check

* OPS-354 revert step condition

Co-authored-by: Sergei Mironov <sergei.mironov@insolar.io>

* PENV-680: remove vendor, update go

* change application structure

* merge

* add burnedAccount

* add burnedAccount

* merge

* merge

* merge

* fix ugorji library

* insolar-scripts revert

Co-authored-by: Alexey Zhunin <zhunin.a.u@gmail.com>
Co-authored-by: Andrey Pavlushev <andrey.pavlushev@insolar.io>
Co-authored-by: Maria Zubareva <maria.zubareva@insolar.io>
Co-authored-by: Andrey Pavlushev <pavlushev@users.noreply.github.com>
Co-authored-by: Andrey Zolin <crrrr@mail.ru>
Co-authored-by: Pavel Moskovkin <34027831+pavel-moskovkin@users.noreply.github.com>
Co-authored-by: Sergei Mironov <sergei.mironov@insolar.io>
Co-authored-by: pavel-moskovkin <you@example.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LinnikD LinnikD LinnikD approved these changes

@MikhailPopov MikhailPopov MikhailPopov approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@pavlushev @codecov-commenter @LinnikD @MikhailPopov