fix: refactor ZeroExInstant state creation logic and fix bug

This commit is contained in:
fragosti
2018-10-18 16:04:30 -07:00
parent 44635f34f0
commit ad6dc8e891
3 changed files with 7 additions and 6 deletions

View File

@@ -29,8 +29,7 @@ export class ZeroExInstant extends React.Component<ZeroExInstantProps> {
additionalAssetMetaDataMap: {},
};
public store: Store;
constructor(props: ZeroExInstantProps) {
super(props);
private static _createPartialStoreStateFromProps(props: ZeroExInstantProps): Partial<State> {
// TODO: Provider needs to not be hard-coded to injected web3.
const assetBuyer = AssetBuyer.getAssetBuyerForStandardRelayerAPIUrl(getProvider(), props.sraApiUrl);
const completeAssetMetaDataMap = {
@@ -42,7 +41,11 @@ export class ZeroExInstant extends React.Component<ZeroExInstantProps> {
selectedAsset: assetUtils.createAssetFromAssetData(props.assetData, completeAssetMetaDataMap),
assetMetaDataMap: completeAssetMetaDataMap,
};
this.store = store.create(storeStateFromProps);
return storeStateFromProps;
}
constructor(props: ZeroExInstantProps) {
super(props);
this.store = store.create(ZeroExInstant._createPartialStoreStateFromProps(this.props));
// tslint:disable-next-line:no-floating-promises
asyncData.fetchAndDispatchToStore(this.store);
}