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

API: Account changes to event stream #3

Closed
spikeekips opened this issue Jun 1, 2018 · 3 comments
Closed

API: Account changes to event stream #3

spikeekips opened this issue Jun 1, 2018 · 3 comments
Assignees
@kfangw @zzim2x
Labels
enhancement New feature or request
Milestone
Opening Testnet

Comments

@spikeekips
Copy link
Contributor

spikeekips commented Jun 1, 2018 •
edited

This api will produce the latest balance of account. If 'text/event-stream' header is set in request header, the node will continuously send the balance of data as response without disconnecting connection.

Basically the 'content-type' of response will be 'application/json' and the data of BlockAccount must be included.
@spikeekips spikeekips added this to the Opening Testnet milestone Jun 1, 2018
@spikeekips spikeekips added the enhancement New feature or request label Jun 1, 2018
This was referenced Jun 4, 2018
Closed
Closed
Closed
Closed
Open
Closed
@leejames00 leejames00 assigned leejames00, kfangw and zzim2x and unassigned leejames00 Jun 5, 2018
@kfangw kfangw mentioned this issue Jun 15, 2018
Merged
@spikeekips spikeekips changed the title API: Account balance changes to event stream API: Account changes to event stream Jun 16, 2018
spikeekips referenced this issue in spikeekips/sebak Jun 17, 2018
@spikeekips
for #3
5c24aa1
@spikeekips
Copy link
Contributor Author

spikeekips commented Jun 17, 2018 •
edited

I wrote some proposal, inspired your EventBroker. spikeekips/sebak@master...spikeekips:proposal-account-changes-event-stream . This branch uses 'github.com/GianlucaGuarini/go-observable' instead of your EventBroker because 'observable' is easy to use :)
I wish my code will help your work. :)

@kfangw
Copy link
Contributor

kfangw commented Jun 18, 2018

@spikeekips thanks spike~ it is very helpful.

@zzim2x zzim2x mentioned this issue Jun 18, 2018
Merged
@kfangw kfangw mentioned this issue Jun 19, 2018
Closed
@kfangw
Copy link
Contributor

kfangw commented Jun 21, 2018

It is solved in this PR
#84

@kfangw kfangw closed this as completed Jun 21, 2018
kfangw referenced this issue in kfangw/sebak Jun 26, 2018
@kfangw
APIs
e6c5fbe 
issue #3 ~ bosnet#7
kfangw referenced this issue in kfangw/sebak Jun 28, 2018
@kfangw
APIs
3224959 
issue #3 ~ bosnet#7
kfangw referenced this issue in kfangw/sebak Jun 28, 2018
@kfangw
APIs
8cbdfde 
issue #3 ~ bosnet#7
kfangw referenced this issue in kfangw/sebak Jun 28, 2018
@kfangw
APIs
0be478c 
issue #3 ~ bosnet#7

remove empty

Fix bug

fix bug

change clients for test. get from httptest server

api8

Info method

marshalJSON

remove useless

fmt

remove useless
kfangw referenced this issue in kfangw/sebak Jun 28, 2018
@kfangw
APIs
f8d48c8 
issue #3 ~ bosnet#7

remove empty

Fix bug

fix bug

change clients for test. get from httptest server

api8

Info method

marshalJSON

remove useless

fmt

remove useless
kfangw referenced this issue in kfangw/sebak Jun 28, 2018
@kfangw
APIs
253332e 
issue #3 ~ bosnet#7

remove empty

Fix bug

fix bug

change clients for test. get from httptest server

api8

Info method

marshalJSON

remove useless

fmt

remove useless
kfangw referenced this issue in kfangw/sebak Jul 2, 2018
@kfangw
APIs
5117ca7 
issue #3 ~ bosnet#7

remove empty

Fix bug

fix bug

change clients for test. get from httptest server

api8

Info method

marshalJSON

remove useless

fmt

remove useless
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kfangw kfangw

@zzim2x zzim2x

Labels
enhancement New feature or request
Projects
None yet
Milestone
Opening Testnet
Development

No branches or pull requests
4 participants
@spikeekips @kfangw @leejames00 @zzim2x