Page MenuHomePhabricator
Differential D6396

[native] Don't crash if Alchemy API key is missing
ClosedPublic
Actions

Authored by ashoat on Jan 26 2023, 9:00 AM.
Tags
None
Referenced Files
F2025426: D6396.diff
Mon, Jun 17, 9:38 AM
Unknown Object (File)
Wed, Jun 12, 10:38 AM
Unknown Object (File)
Tue, Jun 11, 9:34 AM
Unknown Object (File)
Mon, Jun 10, 4:57 PM
Unknown Object (File)
Thu, Jun 6, 7:20 AM
Unknown Object (File)
Wed, Jun 5, 1:50 PM
Unknown Object (File)
Wed, Jun 5, 1:50 PM
Unknown Object (File)
Sat, Jun 1, 2:47 AM
View All 53 Files
Subscribers
None

Details

Reviewers
kamil
• przemek
atul
ginsu
marcin
tomek
Commits
rCOMMbde0aad9e5ec: [native] Don't crash if Alchemy API key is missing
Summary

This fixes ENG-2836. I should've tested this scenario before landing D6373. The docs seemed to indicate that the Alchemy API key was optional, and I know Wagmi "works" without it. But now I think it's only optional for Ethers.js for testnet or something?

Test Plan

Confirm crash no longer occurs when Alchemy API key is missing (no native/facts/alchemy.json file) in native app

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
lib/
components/
4 lines
native/
utils/
8 lines

Diff 21376

View Options

lib/components/ens-cache-provider.react.js

Loading...
View Options

native/utils/ethers-utils.js

Loading...