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

Notify user when a transaction is received #2864

Closed
wants to merge 5 commits into from
Closed

Conversation

ngotchac
Copy link
Contributor

@see #2556

This is a WIP. Right now relies on the tracing mode enabled on the client.
There are no real ways to get accounts' transactions without it and without using a third party as Etherscan...

@ngotchac ngotchac added A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. M5-ui labels Oct 25, 2016
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.003%) to 86.14% when pulling a7bc759 on ng-notify-on-txs into e71c752 on master.


Promise
.all([
this._api.trace.filter({
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are we re-creating the filter with each block? We should just -

  1. create filter
  2. ask for updates on new block

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These are not the same filters as for the eth module. It's just the name of the function, which returns the filtered traces, but doesn't set up a proper filter to get changes from

@ngotchac ngotchac changed the title Notify use when a transaction is received Notify user when a transaction is received Nov 1, 2016
@ngotchac
Copy link
Contributor Author

ngotchac commented Dec 9, 2016

Closed in favor of #3782

@ngotchac ngotchac closed this Dec 9, 2016
@ngotchac ngotchac deleted the ng-notify-on-txs branch December 9, 2016 22:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A3-inprogress ⏳ Pull request is in progress. No review needed at this stage.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants