Skip to content

Commit

Permalink
Automatically merged updates to draft EIP(s) 1193
Browse files Browse the repository at this point in the history
Hi, I'm a bot! This change was automatically merged because:

 - It only modifies existing Draft or Last Call EIP(s)
 - The PR was approved or written by at least one author of each modified EIP
 - The build is passing
  • Loading branch information
ryanio authored and eip-automerger committed Oct 27, 2018
1 parent bf9329d commit e6a686d
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion EIPS/eip-1193.md
Original file line number Diff line number Diff line change
Expand Up @@ -167,14 +167,15 @@ ethereum
.then(subscriptionId => {
subId = subscriptionId;
ethereum.on('notification', result => {
if (result.subscriptionId === subscriptionId) {
if (result.subscription === subscriptionId) {
if (result.result instanceof Error) {
const error = result.result;
console.error(
`Error from newHeads subscription: ${error.message}.
Code: ${error.code}. Data: ${error.data}`
);
} else {
const block = result.result;
console.log(`New block ${block.number}:`, block);
}
}
Expand Down

0 comments on commit e6a686d

Please # to comment.