Skip to content

Commit

Permalink
chore: Remove obsolete keys
Browse files Browse the repository at this point in the history
Remove references to obsolete keys.
* PUBNUB_*: These were used for the old mobile state sync feature,
which was removed a long time ago.
* Etherscan: This was used for incoming transactions, but we've since
switched to our own API.
* Opensea: We've switched to Reservoir
  • Loading branch information
Gudahtt committed Dec 20, 2024
1 parent 021a265 commit 2069269
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 6 deletions.
2 changes: 0 additions & 2 deletions app/scripts/metamask-controller.js
Original file line number Diff line number Diff line change
Expand Up @@ -761,8 +761,6 @@ export default class MetamaskController extends EventEmitter {
}),
});

this.nftController.setApiKey(process.env.OPENSEA_KEY);

const nftDetectionControllerMessenger =
this.controllerMessenger.getRestricted({
name: 'NftDetectionController',
Expand Down
11 changes: 7 additions & 4 deletions builds.yml
Original file line number Diff line number Diff line change
Expand Up @@ -214,13 +214,9 @@ env:
# API keys to 3rd party services
###

- PUBNUB_PUB_KEY: null
- PUBNUB_SUB_KEY: null
- SEGMENT_HOST: null
- SENTRY_DSN: null
- SENTRY_DSN_DEV: null
- OPENSEA_KEY: null
- ETHERSCAN_KEY: null
# also INFURA_PROJECT_ID below

###
Expand Down Expand Up @@ -318,3 +314,10 @@ env:
# This should NEVER be enabled in production since it slows down react
###
- ENABLE_WHY_DID_YOU_RENDER: false

###
# Unused environment variables referenced in dependencies
# Unset environment variables cause a build error. These are set to `null` to tell our build
# system that they are intentionally unset.
###
- ETHERSCAN_KEY: null # Used by `gridplus-sdk/dist/util.js`

0 comments on commit 2069269

Please sign in to comment.