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
Unknown Object (File)
Sat, May 25, 8:00 AM
Unknown Object (File)
Fri, May 24, 2:43 AM
Unknown Object (File)
Sat, Apr 27, 5:20 PM
Unknown Object (File)
Apr 25 2024, 8:09 AM
Unknown Object (File)
Apr 25 2024, 8:09 AM
Unknown Object (File)
Apr 8 2024, 5:12 AM
Unknown Object (File)
Mar 12 2024, 11:37 PM
Unknown Object (File)
Mar 12 2024, 11:36 PM
View All 42 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...