81070fa · bridge · git.iko.soy

git.iko.soy/bridge

Merge pull request #584 from urbit/deps/upgrade

deps: modernization and upgrades

commit81070fa43c4bca7d4f14af28338d0d4f930a8fe5 authortomholford <16504501+tomholford@users.noreply.github.com> dateWed 30 Jun 2021, 15:49 (5 years ago) parent52d428d 885e728

50 files changed+18690−14484

compare with master

unifiedsplit

.eslintrc+1−1 .nvmrc+1 DEVELOPMENT.md+52−2 config-overrides.js−13 jsconfig.json−5 package-lock.json+18353−14253 package.json+39−23 src/Bridge.js+1−2 src/form/Inputs.js+1−1 src/indigo-react/components/Input.js+1−1 src/lib/SolarisClient.js−2 src/lib/authToken.js → src/lib/authToken.ts+9−7 src/lib/constants.js → src/lib/constants.ts+26 src/lib/flowCommand.js → src/lib/flowCommand.ts+1−1 src/lib/keys.js+1−1 src/lib/reticket.js+1−1 src/lib/router.js+1−1 src/lib/tank.js+1−1 src/lib/trezor.js → src/lib/trezor.ts src/lib/txn.js+2−2 src/lib/useIsEclipticOwner.js+1−1 src/lib/usePermissionsForPoint.js+1−1 src/lib/utils/address.test.ts+19 src/lib/utils/address.ts+39 src/lib/validators.js+7−2 src/lib/wallet.js−120 src/lib/wallet.ts+47 src/react-app-env.d.ts+1 src/store/lib/useControlledPointsStore.js+1 src/store/lib/useResidentsStore.js → src/store/lib/useResidentsStore.ts+3−4 src/store/wallet.js+6−5 src/views/AcceptLockup.js+1−1 src/views/Activate/ActivateCode.js+2−1 src/views/Activate/ActivatePassport.js+4−6 src/views/CancelTransfer.js+1−2 src/views/CreateGalaxy.js+1−1 src/views/Login.js → src/views/Login.tsx src/views/Login/Keystore.js+2−1 src/views/Login/Ledger.js → src/views/Login/Ledger.tsx+5−5 src/views/Login/Metamask.js+2−1 src/views/Login/Mnemonic.js+2−1 src/views/Login/PrivateKey.js+3−1 src/views/Login/Ticket.js+2−1 src/views/Login/Trezor.js → src/views/Login/Trezor.tsx+16−7 src/views/Points.js+1−1 src/views/Release/Transfer.js+1−1 src/views/UrbitID/ResetKeys/ResetExecute.js+1−1 src/views/UrbitID/SetProxy.js+2−2 src/views/UrbitOS/NetworkingKeys.js+1−1 tsconfig.json+27

.eslintrc+1−1

@@ -9,6 +9,6 @@

99 "react-hooks"

1010 ],

1111 "rules": {

12− "prettier/prettier": "error"

12+ "prettier/prettier": "warn"

1313 }

1414}

.nvmrcnew file+1

@@ -0,0 +1 @@

1+14.17.0

DEVELOPMENT.md+52−2

@@ -17,8 +17,8 @@ on node 11.1.0, so make sure you're using some other node version.

1717You can use [nvm](https://github.com/creationix/nvm), for example, and do:

1818

1919```

20−$ nvm install 11.0.0

21−$ nvm use v11.0.0

20+$ nvm install 14.17.0

21+$ nvm use v14.17.0

2222```

2323

2424before running `npm run pilot`.

@@ -94,6 +94,56 @@ websocket. Do this by going to `about:config` and setting the

9494Additionally you need to run with the `HTTPS` environment variable set to

9595`true`. Note that `npm run pilot` will handle this automatically.

9696

97+## Testing

98+

99+Automated tests for critical user journeys are coming Soon™️. Until then, here are several manual test cases to consider when making changes. It's worth noting that what to test depends on what the changes are. These test cases have limited utility when you are making a small change on a single screen; however, they are certainly a good idea when touching critical pieces such as transaction sending or secrets derivation.

100+

101+### Mnemonic Login

102+

103+"As a user, I can log into Bridge using a BIP39 mnemonic phrase"

104+

105+1. Click the 'Mnemonic, Metamask, Hardware Wallet ...' Button, then click 'Mnemonic (BIP 39)'

106+2. Paste the mnemonic above (in 'Useful Accounts') into the text area, then click 'Login'

107+3. Confirm that you are able to view the Points list view. Since no points have been created yet, you should see the following message:

108+> No points to display. This wallet is not the owner or proxy for any points.

109+

110+### Planet Creation

111+

112+"As a user, I can create a new planet"

113+

114+1. Once a star has been created, click it in the points list

115+2. Click "Issue Point", enter one of the suggested planet names, and sign and send the transaction

116+3. Log out and back into Bridge, confirm that the newly created planet is rendered in the points list

117+

118+### Invite Issuance

119+

120+"As a user, I can create a new invite"

121+

122+First, in `Bridge.js`, set the `INITIAL_NETWORK_TYPE` to Ropsten:

123+```js

124+const INITIAL_NETWORK_TYPE = NETWORK_TYPES.ROPSTEN;

125+```

126+

127+Then, in `tank.js`, set the port to `3011`:

128+```js

129+const baseUrl = 'https://gas-tank.urbit.org:3011';

130+```

131+

132+1. Ask one of the Collaborators of this project for an Invite code :)

133+

134+(If you're generating a code, don't forget to set `inviteMail.js` port to `:3012` for testnet)

135+

136+### Invite Acceptance

137+

138+"As a new Urbit user, I can set up a new ID with an activation code"

139+

140+1. On the Bridge landing page, click the small "Activate" link

141+2. Paste the Invite code from the previous test case, and click "Go"

142+3. Download the Passport file, then use the enclosed Master Ticket and ID to login to Bridge again

143+### Known Issues

144+

145+When testing the invite acceptance and login flow on Ropsten, some of the Azimuth API calls will fail (e.g., `getConditional`).

146+

97147## Releases

98148

99149To generate a release `bridge-$VERSION.zip` file, use a simple `npm run release`.

config-overrides.jsdeleted−13

@@ -1,13 +0,0 @@

1−const rewireReactHotLoader = require('react-app-rewire-hot-loader');

2−const rewireAliases = require('react-app-rewire-aliases');

3−

4−const alias = rewireAliases.aliasesOptions({

5− 'react-dom': '@hot-loader/react-dom',

6−});

7−

8−module.exports = function override(config, env) {

9− config = rewireReactHotLoader(config, env);

10− config = alias(config, env);

11−

12− return config;

13−};

jsconfig.jsondeleted−5

@@ -1,5 +0,0 @@

1−{

2− "compilerOptions": {

3− "baseUrl": "src"

4− }

5−}

package-lock.json+18353−14253