PaulLaux / eth-hot-wallet

Ethereum wallet with erc20 support / web wallet - built using react, web3, eth-lightwallet
https://eth-hot-wallet.com/
MIT License
300 stars 188 forks source link

Cannot read property 'filter' of undefined #15

Closed kelvinBlockchainTM closed 6 years ago

kelvinBlockchainTM commented 6 years ago

Eth-Hot-Wallet

Please direct redux-saga related questions to stack overflow: http://stackoverflow.com/questions/tagged/redux-saga

10

Description

The check balance button returns an error "Cannot read property 'filter' of undefined" after linking custom token. (Add images if possible)

Steps to reproduce

(Add link to a demo on https://jsfiddle.net or similar if possible)

Versions

PaulLaux commented 6 years ago

Hi kelvinBlockchainTM, Does your token appears on Coinmarketcap? If so, on what # ? Currently the wallet gets the first 100 coins to reduce bandwidth (full price dump is over 1M).

kelvinBlockchainTM commented 6 years ago

I am running my token locally first before on a rinkeby testnet before listing it on coinmarket.

PaulLaux commented 6 years ago

Where does this error appear? Does it prevent the app from loading? To be able to help I need to see the changes. Could you publish the changes to a fork?

kelvinBlockchainTM commented 6 years ago

I have fixed it thanks. Seems the problem was with the abi and token contract address.

kelvinBlockchainTM commented 6 years ago

I am trying to learn some concepts of your code but this error keeps coming up "genKeystore error - TypeError: ks.keyFromPassword is not a function"

PaulLaux commented 6 years ago

What changes did you introduce? Can you publish it?

kelvinBlockchainTM commented 6 years ago

OK. Sure.

On Thu, Mar 15, 2018 at 12:46 PM, Paul Laux notifications@github.com wrote:

What changes did you introduce? Can you publish it?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/PaulLaux/eth-hot-wallet/issues/15#issuecomment-373259643, or mute the thread https://github.com/notifications/unsubscribe-auth/AgempXk26fPiYneNhu8y1Q81P60UlKfnks5tefIwgaJpZM4SiMzq .

--

https://ms-my.facebook.com/blockchaintmhub/ https://www.linkedin.com/company/blockchaintmhub https://www.instagram.com/blockchaintmhub/

KELVIN NDEREBA NGENDO

BLOCKCHAIN DEVELOPER

BTM Blockchain Technology (M) Sdn Bhd

Wisma Shell Building, 2nd Floor, Left Wing 3450,

Jalan Teknokrat 3, 63000 Cyberjaya, Selangor, Malaysia

M: +6013- 981 6191 <+60%2019-506%202665>

E: kelvin@blockchaintmhub.io syazana@blockchaintmhub.io

W: blockchaintmhub https://blockchaintmhub.io/.io

kelvinBlockchainTM commented 6 years ago

Kindly Take a look https://github.com/kelvinBlockchainTM/ODIN-API/tree/test

On Thu, Mar 15, 2018 at 12:48 PM, Kelvin Ndereba kelvin@blockchaintmhub.io wrote:

OK. Sure.

On Thu, Mar 15, 2018 at 12:46 PM, Paul Laux notifications@github.com wrote:

What changes did you introduce? Can you publish it?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/PaulLaux/eth-hot-wallet/issues/15#issuecomment-373259643, or mute the thread https://github.com/notifications/unsubscribe-auth/AgempXk26fPiYneNhu8y1Q81P60UlKfnks5tefIwgaJpZM4SiMzq .

--

https://ms-my.facebook.com/blockchaintmhub/ https://www.linkedin.com/company/blockchaintmhub https://www.instagram.com/blockchaintmhub/

KELVIN NDEREBA NGENDO

BLOCKCHAIN DEVELOPER

BTM Blockchain Technology (M) Sdn Bhd

Wisma Shell Building, 2nd Floor, Left Wing 3450,

Jalan Teknokrat 3, 63000 Cyberjaya, Selangor, Malaysia

M: +6013- 981 6191 <+60%2019-506%202665>

E: kelvin@blockchaintmhub.io syazana@blockchaintmhub.io

W: blockchaintmhub https://blockchaintmhub.io/.io

--

https://ms-my.facebook.com/blockchaintmhub/ https://www.linkedin.com/company/blockchaintmhub https://www.instagram.com/blockchaintmhub/

KELVIN NDEREBA NGENDO

BLOCKCHAIN DEVELOPER

BTM Blockchain Technology (M) Sdn Bhd

Wisma Shell Building, 2nd Floor, Left Wing 3450,

Jalan Teknokrat 3, 63000 Cyberjaya, Selangor, Malaysia

M: +6013- 981 6191 <+60%2019-506%202665>

E: kelvin@blockchaintmhub.io syazana@blockchaintmhub.io

W: blockchaintmhub https://blockchaintmhub.io/.io

kelvinBlockchainTM commented 6 years ago

screenshot from 2018-03-15 13-27-28

PaulLaux commented 6 years ago

This is an error coming from saga.js in app/containers/homepage/saga.js

Somewhere in the function genKeystore()

Add prints to console from this function to see why is ks not defined.

If you can't resolve let me know and I'll check your code on the following days.

kelvinBlockchainTM commented 6 years ago

We are working on enhancing your product and using it in our app , so we need to create API endpoints based on your app. We need your help in finding out how to achieve it. Reach me on nderebakelvin@gmail.com