search

trufflesuite / drizzle

Reactive Ethereum dapp UI suite
907 stars 238 forks source link

Fix: accounts is not merged into store as array #114

Open Velissel opened 3 years ago

Velissel commented 3 years ago 
if (action.type === AccountsActions.ACCOUNTS_FETCHED) {
    return Object.assign({}, state, action.accounts)
}

at packages/store/src/accounts/accountsReducer.js

action.accounts is an array, but it is merged int as object. According to other components, I assume array should be the correct data type.

Velissel commented 3 years ago

how do I submit my fix branch to create PR? I am always denied with 403