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

MN-655: added transaction type #51

Merged
merged 6 commits into from
Aug 4, 2020
Merged

Conversation

pavlushev
Copy link
Contributor

what I did:

  • added a transaction type to the TransferToDeposit method for observer collectors.
    In collectors, we can understand where the call came from and determine the appropriate type of transaction.

uhzxl and others added 3 commits July 31, 2020 13:02
* MN-651: split spec files

* MN-651: return temporary admin methods in api-exported.yaml
@codecov-commenter
Copy link

codecov-commenter commented Aug 3, 2020

Codecov Report

❗ No coverage uploaded for pull request base (MN-574-1@694864f). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##             MN-574-1      #51   +/-   ##
===========================================
  Coverage            ?   14.95%           
===========================================
  Files               ?        7           
  Lines               ?     1070           
  Branches            ?        0           
===========================================
  Hits                ?      160           
  Misses              ?      897           
  Partials            ?       13           

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 694864f...0d67716. Read the comment docs.

MikhailPopov and others added 2 commits August 4, 2020 20:03
* MN-662: fix TestMigrationDaemonTransferDeposit

* MN-662: fix TransferDeposit tests
� Conflicts:
�	application/api/spec/api-exported-internal.yaml
�	application/api/spec/api-exported.yaml
�	vendor/github.com/ugorji/go/codec/LICENSE
@pavlushev pavlushev merged commit 62ce17e into MN-574-1 Aug 4, 2020
@pavlushev pavlushev deleted the MN-655-added-transaction-type branch August 4, 2020 17:34
pavlushev added a commit that referenced this pull request Aug 10, 2020
* 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-655: added transaction type (#51)

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

* MN-651: split spec files

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

* MN-655: added transaction type

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

* MN-655: rename TransferType.go to the transfer_type.go

* MN-662: fix TestMigrationDaemonTransferDeposit (#52)

* MN-662: fix TestMigrationDaemonTransferDeposit

* MN-662: fix TransferDeposit tests

Co-authored-by: Alexey Zhunin <zhunin.a.u@gmail.com>
Co-authored-by: Mikhail Popov <53470434+MikhailPopov@users.noreply.github.com>

Co-authored-by: Alexey Zhunin <zhunin.a.u@gmail.com>
Co-authored-by: Maria Zubareva <maria.zubareva@insolar.io>
Co-authored-by: Mikhail Popov <53470434+MikhailPopov@users.noreply.github.com>
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.

7 participants