Skip to content
This repository has been archived by the owner on Apr 15, 2020. It is now read-only.

Changes requested in PR #10 #12

Merged
merged 46 commits into from Dec 11, 2018
Merged

Changes requested in PR #10 #12

merged 46 commits into from Dec 11, 2018

Conversation

susruth
Copy link
Contributor

@susruth susruth commented Dec 9, 2018 •
edited

Description

Architecture level changes requested in the PR #10

Motivation

Cleaner and more understandable architecture.

Design

  • Replace Swapperd driver with Compose driver
  • Two foundations: Swap, Blockchain
  • All components use Run to begin a for loop and accept channels as arguments to Run (done, requests, responses)
  • Rename status.Book to status.Statuses
  • Introduce balance.Balances
  • Rename fund.Manager to wallet.Wallet
  • Move adapter/router to core/router

Related Work
PR #11
PR #13

Susruth Nadimpalli and others added 30 commits November 30, 2018 06:38
core/router: moved router to core
e7f5311
adapter/server: moved server interface from router to server adapter
38d1461
adapter/db: updated adapter to work with the new router
490f075
driver/composer: replaced swapperd with composer
cbf2c79
core/balance: add balance core service
611296f
adapter/server: updated get balances handler to use cache
a8e04c3
cmd/swapperd-unix,cmd/swapperd-win: updated to use composer
611f649
cmd/install.sh: fix the timestamp when doing config backup
71a045b
cmd/install.sh: make the backup format consistent when installing and…
194bb7a 
… removing
cmd/installer: updated installer to create multiple keystores
d8e97e4
cmd/swapperd-win: updated windows swapperd to support both networks
3792fcf
cmd/swapperd: updated unix swapperd to support both networks
fb7bc41
driver/composer: updated composer to use balances
00536a8
driver/keystore: moved new mnemonic generation to installer
3f81101
driver/leveldb: updated leveldb to support multiple networks
b0977e6
Merge branch 'composer' of https://github.com/republicprotocol/swapperd…
fede8f8 
… into composer
adapter/fund: add contexts with timeout
58c5d11
core: use go routenes to write responses to queries
21378d2
adapter/server, driver/keysore: updated to use bcrypt
676cc5f
adapter/fund: fixed verify balance function
62642ed
updated gopkg lock
eecf861
driver/keystore: updated mainnet keystore to use ethereum mainnet
c7c3e7f
cmd/installer, driver/leveldb: updated windows path
5ac7f63
cmd/installer: updated installer to use the new windows paths
d833e49
driver/composer: updated to composer to accept home directory
00e796a
cmd: updated binaries to use the new composer
6298a8b
driver/keystore: updated keystore to accept home directory location
be2b3ee
driver/leveldb: updated leveldb to accept home dir location
347a27e
cmd: add install script for swapperd tests
42e623c
core/status, driver/composer: renamed status book to statuses
297c6e8
Susruth Nadimpalli added 15 commits December 10, 2018 10:22
adapter/wallet: renamed fund.Manager to wallet.Wallet
d5aa788
adapter/binder: use wallet.Wallet instead of fund.Manager
ff2f120
adapter/server: use wallet.Wallet instead of fund.Manager
3560d9a
cmd/installer: use wallet.Wallet instead of fund.Manager
0026301
driver/composer: use wallet.Wallet instead of fund.Manager
bc9c93e
driver/keystore: use wallet.Wallet instead of fund.Manager
ec2f62a
foundation: split foundation into swap and blockchain
8bb3048
adapter/binder: updated to use Swap Blobs instead of Swap Requests
90fe7b1
core/router: removed router in favour of bootloader
679b996
core/bootload: add bootloader that loads historical and pending swaps
814d4d0
adapter/server: add post bootload functionality
4c31c9f
core/swapper: add storage to swapper to modify swaps on change
dc77ac4
foundation/swap: removed swap request and swap result
e68dc28
adapter/db: updated db to support the new storage interface
1d7d749
driver/composer: updated composer to use bootloader
a2fb445
@susruth susruth requested a review from loongy December 10, 2018 11:47
@susruth susruth self-assigned this Dec 11, 2018
@loongy
Copy link
Contributor

loongy commented Dec 11, 2018

Is the work in this PR included in #11?

@susruth
Copy link
Contributor Author

susruth commented Dec 11, 2018

Yes, but once this PR is merged. #11 will only show the differences between this and fees branch.

@loongy loongy merged commit 85ea9f4 into master Dec 11, 2018
@tok-kkk tok-kkk deleted the composer branch December 17, 2018 03:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@loongy loongy Awaiting requested review from loongy

Assignees

@susruth susruth

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@susruth @loongy