Polkadot identicons

[Tbaut]

• bring in @polkadot/reactnative-identicon and the needed wasm deps. Striped down as much as I could. The building time is actually ok the second time (still last a couple min the first time)

• Set --max_old_space_size=8192 in scripts to avoid out of memory problem

• upgrade react-native (0.60.4), bignumber.js, hoist-non-react-statics

• remove some unused deps: react-native-simple-picker, yarn

• added a sample of identicon, should be removed before merging. It should look like:
  

[hanwencheng]

find one jest related problem on iOS, which block the building process:

Solved with root directory specification in jest.config.js

following is a simple config file:

module.exports = {
  verbose: true,
  roots: ['<rootDir>/src'],
};

[Tbaut]

ok I was able to create a signed and fully working apk.. although it takes ages, the building of the metro bundle is long the first time but quite snappy after.

@hanwencheng if you say it's working well on iphone, I'd remove my identicon samples and merge.

[hanwencheng]

Got error on iOS

I have tried clear all the mentioned stuff, and the ios build folder, also uninstall the app. It seems the crypto package is not indexed.

[hanwencheng]

As we are not using nodify, does these Buffer related code still needed?

[pmespresso]

+1, IIRC the shim.js was generated by the nodify --HACK script right?

[Tbaut]

Good point. It seems that process.browser = true; is still needed though. I removed the rest and things still work.. on android at least 👍

(I get a "process.split undefined" if I remove the process.browser = true;)

[Tbaut]

As we talked about in person with @hanwencheng, it seems that the 'crypto' override done here isn't picked-up correctly.

I'd advise to try in this order:

• pull the lastest changes and try again, I removed the crypto import in shim.js, this might help.

if things still don't work:

• make sure to do a yarn clean, and that all the commands in there work properly (install watchman if needed).

if things still don't work:

• play with babel.rc instead of babel.config.js and the rewrite require plugin. Have a look here https://github.com/polkawallet-io/polkawallet-RN/pull/142/files, it seems to work for them.

[hanwencheng]

LGTM, successful tested on iOS simulator

[pmespresso]

this isn't a shim anymore, so maybe just set process.browser = true directly in App.js now?

[Tbaut]

🤷‍♂️ happy to remove that.

[Tbaut]

Copying it in Apps throws

undefined is not and object (evaluating process.version.split)

Not renaming it after discussion because it still is a black magic shim.

[pmespresso]

looks good to me, just minor comments